-
-
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 #12379: Checker false violation removals on other failures #12725
Conversation
.ci/checker-framework.groovy
Outdated
@@ -112,7 +112,8 @@ private static int checkCheckerFrameworkReport(final String profile) { | |||
*/ | |||
private static List<List<String>> getCheckerFrameworkErrors(final String profile) { | |||
final List<String> checkerFrameworkLines = new ArrayList<>() | |||
final String command = "mvn -e --no-transfer-progress clean compile -P${profile}" | |||
final String command = "mvn -e --no-transfer-progress clean compile" | |||
+ " -P${profile},no-validations" |
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.
CI is failing, +
has to be on previous line. (It is a groovy thing)
https://github.com/checkstyle/checkstyle/actions/runs/4097261782/jobs/7065502375#step:6:15
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.
Yes, we should disable this rule one day in future.
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.
My understanding was that this is something in groovy, not a check we turned on.
https://stackoverflow.com/a/5079914/1016482
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.
I rebased and fixed the code.
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.
thanks a lot, I just pushed a fix and got surprised that I have no PR
e3915b2
to
f93c5ce
Compare
CI is all green. https://github.com/checkstyle/checkstyle/actions/runs/4116348792/jobs/7106284737#step:6:64 |
fixes #12379