-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue #11734: Add codenarc to CI for groovy files #11739
Conversation
fa9fddf
to
7941900
Compare
I would move this check to Travis or other not high demand CI. It it most likely that such script will never be updated after summer, so better to have it run in some less critical CI. But we can do this move at end of summer |
5e3e546
to
84c4ffa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Items:
689e450
to
618c38a
Compare
dbd55ac
to
ce4ba54
Compare
UnnecessaryGroovyImport { | ||
doNotApplyToFileNames = 'codenarc.groovy' | ||
} | ||
UnusedImport { | ||
doNotApplyToFileNames = 'codenarc.groovy' | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
False positives, those imports are used.
ce4ba54
to
3ed6748
Compare
@Vyom-Yadav see CI failure at https://cloud.drone.io/checkstyle/checkstyle/10876/5/2 |
3ed6748
to
f1c2717
Compare
Done. |
Resolves #11734
Failing CodeNarc analysis (Proof of working): https://github.com/checkstyle/checkstyle/runs/7021070841?check_suite_focus=true