fluttercommunity / fluttercommunity/plus_plugins
[Request]: Run flutter analyze workflow in proper scopes and enforce its results
Open
enhancement
triage
- Dominant language
- Dart
- Stars
- 1.9k
- Forks
- 1.3k
- PR merge metrics
- No merged PRs in 30d
Description
### Plugin
All
### Use case
It makes no sense to run `analyze` on all packages in every PR.
Here is an example where in PR with `device_info_plus` files changes only I see warnings for another plugin, because the analyze task runs for all plugins:
### Proposal
Suggest to do multiple changes:
- Run `analyze` only in the scope of changed files in PRs.
- Make analyzer warnings failing the set of CI checks to enforce issues resolutions, instead of catching up with fixes later as we do now.
Contributor guide
Assessment
This issue has not been assessed yet.