Skip to content

Conversation

@Hixie
Copy link
Contributor

@Hixie Hixie commented Mar 17, 2024

This is part 5 of a broken down version of the #140101 refactor.

This PR adds a README to flutter_goldens with breadcrumbs to other sources of documentation.

@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact "@test-exemption-reviewer" in the #hackers channel in Chat (don't just cc them here, they won't see it! Use Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@github-actions github-actions bot added a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels. labels Mar 17, 2024
@Hixie
Copy link
Contributor Author

Hixie commented Mar 17, 2024

test-exempt: documentation

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fly-by comment without looking into this any further: Should this be README.md to get full markdown formatting in these readmes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I meant to call it that. Good catch.

This PR adds a README to flutter_goldens with breadcrumbs to other sources of documentation.
@Hixie Hixie force-pushed the golden_refactor_journey_6 branch from 3afbaa4 to 37a5888 Compare March 26, 2024 20:49
@Piinks Piinks self-requested a review March 28, 2024 22:02
Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Hixie Hixie added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 29, 2024
@auto-submit auto-submit bot merged commit 887bdcf into flutter:master Mar 29, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Mar 30, 2024
flutter/flutter@8528881...d12ba5c

2024-03-30 [email protected] Roll Flutter Engine from 08eb5ad6e498 to 3588d31a98f6 (1 revision) (flutter/flutter#146031)
2024-03-30 [email protected] Roll Flutter Engine from df581c316485 to 08eb5ad6e498 (2 revisions) (flutter/flutter#146028)
2024-03-30 [email protected] Roll Flutter Engine from 5df1042cd927 to df581c316485 (1 revision) (flutter/flutter#146025)
2024-03-30 [email protected] Roll Flutter Engine from dce6ce366c74 to 5df1042cd927 (2 revisions) (flutter/flutter#146024)
2024-03-30 [email protected] Roll Flutter Engine from 221b49ae4a82 to dce6ce366c74 (1 revision) (flutter/flutter#146023)
2024-03-30 [email protected] Roll Flutter Engine from 0a751669e722 to 221b49ae4a82 (1 revision) (flutter/flutter#146022)
2024-03-30 [email protected] Roll Flutter Engine from 8ec35b6d63ba to 0a751669e722 (2 revisions) (flutter/flutter#146020)
2024-03-29 [email protected] Roll Flutter Engine from 4b6836f8ef00 to 8ec35b6d63ba (3 revisions) (flutter/flutter#146014)
2024-03-29 [email protected] Roll Flutter Engine from 4c079a6ff502 to 4b6836f8ef00 (3 revisions) (flutter/flutter#146010)
2024-03-29 [email protected] Implementing switch expressions in `flutter_tools/` (flutter/flutter#145632)
2024-03-29 [email protected] Generate test metrics consistently. (flutter/flutter#145943)
2024-03-29 [email protected] Roll Flutter Engine from 32c9dab552f0 to 4c079a6ff502 (3 revisions) (flutter/flutter#146007)
2024-03-29 [email protected] Add flutter_goldens README (flutter/flutter#145278)
2024-03-29 [email protected] Remove state shared across tests (flutter/flutter#145281)
2024-03-29 [email protected] Roll Flutter Engine from 8480145cc39d to 32c9dab552f0 (1 revision) (flutter/flutter#146005)
2024-03-29 [email protected] InputDecorator M3 tests migration - Step7 - container (flutter/flutter#145583)
2024-03-29 [email protected] Roll Flutter Engine from 7176173ea303 to 8480145cc39d (3 revisions) (flutter/flutter#146001)
2024-03-29 [email protected] Roll Flutter Engine from b16c0f136cdd to 7176173ea303 (1 revision) (flutter/flutter#145994)
2024-03-29 [email protected] Upgrade leak_tracker. (flutter/flutter#145997)
2024-03-29 [email protected] Roll Flutter Engine from 68aa9ba386e1 to b16c0f136cdd (10 revisions) (flutter/flutter#145990)
2024-03-29 [email protected] Roll Packages from 924c7e6 to 51faaa1 (8 revisions) (flutter/flutter#145986)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants