-
Notifications
You must be signed in to change notification settings - Fork 700
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
Sort 'All linter rules' alphabetically #5124
Comments
Thanks for reporting this! It makes sense for them to be alphabetical, but I never considered this way of determining what lints are new. My hope is they are consistently documented in the SDK changelog. We've also been discussing introducing a new index/discovery page for lints and that could perhaps have a way to sort by release (among other options). |
I believe the older page did suggest to copy the entire list to analysis_options.yaml and then remove unwanted ones. That's the reason it was and still is a piece of valid YAML, and not just a list. I have the same workflow of copying it. So this impacts someone like me. |
I really appreciate that feedback and insight! If you have any other feedback on linter rules, discovering them, or their documentation please let us know. We're beginning to plan work around improving the linter docs currently :) Until the fix (#5126) is approved and merged, the staged version of the change should have what you're looking for. For additional reference, I believe the two new lints in Dart 3.1 are |
Page URL
https://dart.dev/tools/linter-rules/all.html
Page source
https://github.com/dart-lang/site-www/tree/main/src/tools/linter-rules/all.md
Describe the problem
Lints should be sorted alphabetically to simplify diffing when upgrading to a newer SDK.
Lints were sorted on the older page that was here: https://dart-lang.github.io/linter/lints/options/options.html
Expected fix
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: