fix unsorted directives #80901
Merged
fix unsorted directives #80901
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
flutter-dashboard
bot
added
a: internationalization
dart-bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Apr 22, 2021
Mitigating PRs: https://dart-review.googlesource.com/c/sdk/+/196223 https://dart-review.googlesource.com/c/sdk/+/196180 flutter/flutter#80817 flutter/flutter#80901 Change-Id: Ib916d541076e24b9e8db1d6120dbc043d9e40b87 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/196026 Commit-Queue: Phil Quitslund <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]>
dart-bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Apr 23, 2021
This reverts commit 17644ee. Reason for revert: This caused various failures in HHH configuration which is an early warning that dart-lang/sdk -> flutter/{engine,flutter} rolls will not work smoothly. See more information in go/dartvm-gardening-log for various failures this causes. Original change's description: > linter 1.4.0 > > Mitigating PRs: > > https://dart-review.googlesource.com/c/sdk/+/196223 > https://dart-review.googlesource.com/c/sdk/+/196180 > > flutter/flutter#80817 > flutter/flutter#80901 > > > > > > Change-Id: Ib916d541076e24b9e8db1d6120dbc043d9e40b87 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/196026 > Commit-Queue: Phil Quitslund <[email protected]> > Reviewed-by: Brian Wilkerson <[email protected]> [email protected],[email protected],[email protected] Change-Id: I15af9de53d98de7b65902a015bef7ac2d368abeb No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/196499 Reviewed-by: Martin Kustermann <[email protected]> Commit-Queue: Martin Kustermann <[email protected]>
This was referenced Apr 23, 2021
Merged
dart-bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Apr 27, 2021
This is a reland of 17644ee Additional supporting changes in flutter (and plugins): flutter/flutter#81017 flutter/plugins#3823 flutter/flutter#80817 flutter/flutter#80901 flutter/flutter#81210 Additional context in meta linter issue: https://github.com/dart-lang/linter/issues/2609 Original change's description: > linter 1.4.0 > > Mitigating PRs: > > https://dart-review.googlesource.com/c/sdk/+/196223 > https://dart-review.googlesource.com/c/sdk/+/196180 > > flutter/flutter#80817 > flutter/flutter#80901 > > > > > > Change-Id: Ib916d541076e24b9e8db1d6120dbc043d9e40b87 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/196026 > Commit-Queue: Phil Quitslund <[email protected]> > Reviewed-by: Brian Wilkerson <[email protected]> Change-Id: Ie6b9682a2f54ef039dd7ac9c6a46cb925615acef Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/196581 Commit-Queue: Phil Quitslund <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The upcoming linter release includes a fix to sort_directives that will trigger a host of new violations. This fixes issues that cropped up since #80817
See: https://dart-review.googlesource.com/c/sdk/+/196026
Failures: https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8849308566291597920/+/steps/analyze_flutter_flutter/0/stdout
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.