Open chykon opened 1 year ago
This is a cool tool, good idea!
What are your thoughts on when this check should be run? Suppose new rules show up and start blocking all open PRs on the same set of new lint issues even though no particular PR was the root-cause for the PR to fail. Worse, imagine that multiple people resolve the new lint issues in their PRs in slightly different ways, causing merge conflict headaches for everyone after the first PR to get merged.
I'd almost wish that some periodic automation would create a new independent PR to update lints any time it detects new lint rules created. Then it would be up to others to try to help fix the PR failures before it can get merged in. I know its possible to set up cron-triggered GitHub actions, but I'm not that familiar with automation to create pull requests. Maybe we'd need a separate lints
branch of some sort in the main intel/rohd
repo which others could open PRs against to fix lint issues? I don't know, open to ideas and thoughts.
Indeed, I did not think about possible conflicts in different PRs. It's good that you pointed this out.
I like your suggestion for a way to resolve conflicts and check for new rules.
_Reminder: a local check (run_checks.sh
) should also not encourage conflicting changes (make checking for new lints optional?)._
Description & Motivation
Since the lints for Dart is updated quite often and no official package that provides a complete set of rules, the problem of automating tracking arises. This pull request contains changes to address this issue without using third-party dependencies.
It is proposed to add a step to check for new rules in
CI
and in thetool/run_checks.sh
script.ISSUES
After updating
analysis_options.yaml
, new rules-related issues have been identified:The first rule has been temporarily disabled due to file header peculiarities (https://github.com/intel/rohd/issues/264). For the other two, changes will be provided in a separate PR.
Related Issue(s)
https://github.com/intel/rohd/issues/264
Testing
See step
Check Dart lints
:Backwards-compatibility
Yes, because of the new rules,
dart analyze
throws an error.Documentation
No.