-
Notifications
You must be signed in to change notification settings - Fork 688
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
SONARJAVA-4973 Fix FP on S1118 with Lombok @UtilityClass in automatic analysis #4960
SONARJAVA-4973 Fix FP on S1118 with Lombok @UtilityClass in automatic analysis #4960
Conversation
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.
We have the substance here, let's try and fix the form
java-checks/src/main/java/org/sonar/java/filters/LombokFilter.java
Outdated
Show resolved
Hide resolved
java-checks/src/main/java/org/sonar/java/filters/LombokFilter.java
Outdated
Show resolved
Hide resolved
java-checks/src/main/java/org/sonar/java/filters/LombokFilter.java
Outdated
Show resolved
Hide resolved
java-checks/src/main/java/org/sonar/java/filters/LombokFilter.java
Outdated
Show resolved
Hide resolved
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.
We should use some formatting before committing this new 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.
I did reformat the code in the IDE. Please let me know if you think my IDE is misconfigured.
java-checks/src/main/java/org/sonar/java/filters/LombokFilter.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Dorian Burihabwa <[email protected]>
Co-authored-by: Dorian Burihabwa <[email protected]>
Co-authored-by: Dorian Burihabwa <[email protected]>
Co-authored-by: Dorian Burihabwa <[email protected]>
Co-authored-by: Dorian Burihabwa <[email protected]>
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 for going back to this. I see that that is not easy to avoid duplication in the logic without additional parameters or impacting the testing of more annotation.
105c70c
to
55de6e9
Compare
Quality Gate passedIssues Measures |
https://sonarsource.atlassian.net/browse/SONARJAVA-4973