-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[go_router_builder] fixed typo in README #3835
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Replaced "were" in line 344 of the README with "where".
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, you will need to sign CLA in order to contribute to go_router
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@chunhtai When adding override labels, please leave a comment explaining why, per the last paragraphs here and here. Since overriding for READMEs is the opposite of the documented policy it should include a clear explanation of why. (That's not to say that overriding it is necessariy incorrect in this instance, just that it should be explained.) |
@TheUltimateOptimist In the future please don't delete the checklist when submitting PRs; you skipped several steps in it. |
override version and change logs The changes to the readme is insignificant enough to not affect the readability. the change also doesn't worth mentioning in change logs since it doesn't affect developers |
auto label is removed for flutter/packages, pr: 3835, due to - The status or check suite repo_checks has failed. Please fix the issues identified (or deflake) before re-applying this label. |
flutter/packages@6bd59cd...a0f8fd8 2023-05-05 [email protected] tree fix (flutter/packages#3910) 2023-05-05 [email protected] manual roll and fix (flutter/packages#3903) 2023-05-04 [email protected] [file_selector] Add `getDirectoryPaths` (flutter/packages#3871) 2023-05-04 [email protected] [go_router_builder] fixed typo in README (flutter/packages#3835) 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-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Replaced "were" in line 344 of the README with "where".
Replaced "were" in line 344 of the README with "where".