fluttercommunity / fluttercommunity/plus_plugins

[Request]: Run flutter analyze workflow in proper scopes and enforce its results

Open
#2,775 0 comments 0 reactions 0 assignees View on GitHub
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:
Screenshot 2024-03-30 at 15 13 29

### 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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.