Sort reporter options #2475
Merged
Sort reporter options #2475
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
Previous versions of `package:args` alphabetically sorted the `allowedHelp` entries for help output. The latest release removes the sorting and breaks the change-detection tests for the help output. dart-lang/core#852 Manually sort the keys of the reporters map to match the alpha sort of the old output.
PR Health |
jakemac53
approved these changes
Mar 31, 2025
copybara-service bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Apr 2, 2025
Revisions updated by `dart tools/rev_sdk_deps.dart`. core (https://github.com/dart-lang/core/compare/61e6771..a6e81e0): a6e81e0b 2025-03-31 Lasse R.H. Nielsen Make URL strategy better at recognizing URLs. (dart-lang/core#873) 379e9c2e 2025-03-25 dependabot[bot] Bump dart-lang/setup-dart from 1.7.0 to 1.7.1 in the github-actions group (dart-lang/core#869) dartdoc (https://github.com/dart-lang/dartdoc/compare/d400676..cbc1885): cbc18857 2025-03-25 Sam Rawlins Improve the error message when an import cannot be resolved (dart-lang/dartdoc#4023) 18a7631d 2025-03-25 Parker Lougheed Place footer below main content (dart-lang/dartdoc#4014) f41bd3ab 2025-03-24 Sam Rawlins Improve failure when import cannot be found (dart-lang/dartdoc#4022) ecosystem (https://github.com/dart-lang/ecosystem/compare/2317263..391a80c): 391a80c 2025-04-01 Brian Wilkerson Update prompts.dart (dart-lang/ecosystem#349) 6a89899 2025-04-01 Brian Wilkerson Update create_tuning_data.dart (dart-lang/ecosystem#350) http (https://github.com/dart-lang/http/compare/1b6e28d..e988925): e988925 2025-03-31 Brian Quinlan Remove .gitignored files (dart-lang/http#1728) 29c5733 2025-03-25 Brian Quinlan Improve the reliability of cronet_http/ok_http workflows (dart-lang/http#1736) dd56b86 2025-03-21 Brian Quinlan Remove cronet_http.impl (dart-lang/http#1734) 7f5ad18 2025-03-21 Brian Quinlan Adding missing license headers (dart-lang/http#1735) fedead7 2025-03-20 Brian Quinlan [cronet_http] Prepare to release 1.3.3 (dart-lang/http#1733) i18n (https://github.com/dart-lang/i18n/compare/d9cce0b..2701040): 2701040 2025-04-01 dependabot[bot] Bump the github-actions group with 2 updates (dart-lang/i18n#959) 5101533 2025-04-01 Moritz Fix hashes.dart formatting (dart-lang/i18n#960) shelf (https://github.com/dart-lang/shelf/compare/2af8529..082d3ac): 082d3ac 2025-04-01 dependabot[bot] Bump actions/cache from 4.2.2 to 4.2.3 in the github-actions group (dart-lang/shelf#475) test (https://github.com/dart-lang/test/compare/9e349d0..c1fa1e6): c1fa1e69 2025-04-01 dependabot[bot] Bump the github-actions group with 3 updates (dart-lang/test#2476) 31a2d64c 2025-03-31 Nate Bosch Sort reporter options (dart-lang/test#2475) webdev (https://github.com/dart-lang/webdev/compare/302b6db..697f2f7): 697f2f7f 2025-03-24 Jessy Yameogo renamed DWDS injector parameter (dart-lang/webdev#2602) af33df71 2025-03-21 Tristan Ross Add offline mode (dart-lang/webdev#2483) 5c805f14 2025-03-21 Jessy Yameogo Updated DWDS to include a boolean flag that enables debugging support when set to true (dart-lang/webdev#2601) Change-Id: I7af88306b77efa3dab2bdef0d7543663317448f1 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/419682 Auto-Submit: Devon Carew <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Previous versions of
package:argsalphabetically sorted theallowedHelpentries for help output. The latest release removes thesorting and breaks the change-detection tests for the help output.
dart-lang/core#852
Manually sort the keys of the reporters map to match the alpha sort of
the old output.