You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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:
analyze
only in the scope of changed files in PRs.The text was updated successfully, but these errors were encountered: