Repository navigation
Make analysis_options in flutter/plugins much stricter #76229
Description
Currently the flutter/plugins analysis_options.yaml is essentially just https://github.com/google/pedantic/blob/master/lib/analysis_options.1.8.0.yaml. Ideally we should be using all (or at the very least most) of the same options as flutter/flutter.
We'd need to figure out how to introduce these it a way that wouldn't be incredibly disruptive to all future incremental changes. Likely turning them on one by one in conjunction with repo-wide fixes for everything they turn up (one PR per lint option if there are widespread changes).
Activity
stuartmorgan-g commentedon Mar 11, 2021
To get a sense of the scale of the issue, I dropped flutter/flutter's options into the plugins repo; the result is about 5,000 issues. Even going option by option, there are some options with many hundreds of instances (e.g., preferring single quoted strings was ~800).
Instead I think we'll need to this plugin by plugin (and for large plugins, maybe even option by option within the plugin). That will also make it easier to deal with versioning/changelogs, since some changes may be potentially behavioral and thus require publishing.
Prep for alignment with Flutter analysis options
Prep for alignment with Flutter analysis options (#3703)
[path_provider] Switch to new analysis options (#3755)
Switch script/tools over to the new analysis options (#3777)
[path_provider] Switch to new analysis options (flutter-team-archive#…
Switch script/tools over to the new analysis options (flutter-team-ar…
Remove some trivial custom analysis options files (#4379)
48 remaining items
[various] Enable use_build_context_synchronously (flutter-team-archiv…
Prep for alignment with Flutter analysis options (#3703)
Switch script/tools over to the new analysis options (#3777)
Re-sync analysis_options.yaml with flutter/flutter (#5695)
[various] Enable use_build_context_synchronously (flutter#6585)
Remove some trivial custom analysis options files (flutter#4379)
[file_selector] Remove custom analysis options (flutter#4382)
[quick_actions] Migrate to new analysis options (flutter#4548)
[url_launcher] Switch to new analysis options (flutter#4551)
[local_auth] Switch to new analysis options (flutter#4604)
[camera] Switch platform interface to new analysis options (flutter#4613
Re-sync analysis_options.yaml with flutter/flutter (flutter#5695)
github-actions commentedon Mar 4, 2023
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of
flutter doctor -vand a minimal reproduction of the issue.