-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[java] CommentDefaultAccessModifier - add co.elastic.clients.util.VisibleForTesting as default suppressed annotation #4285
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.
Thanks for the PR!
However, in the current state, we can't merge it: Please restore the unintended removal of ParameterizedTest
.
Also I'd use a more accurate test description.
Thanks!
...main/java/net/sourceforge/pmd/lang/java/rule/codestyle/CommentDefaultAccessModifierRule.java
Outdated
Show resolved
Hide resolved
.../resources/net/sourceforge/pmd/lang/java/rule/codestyle/xml/CommentDefaultAccessModifier.xml
Outdated
Show resolved
Hide resolved
3d24cb0
to
918aa04
Compare
suppressed annotations for rule CommentDefaultAccessModifierRule
Co-authored-by: Andreas Dangel <andreas.dangel@adangel.org>
918aa04
to
f6504a6
Compare
Generated by 🚫 Danger |
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, LGTM!
[java] CommentDefaultAccessModifier - add co.elastic.clients.util.VisibleForTesting as default suppressed annotation #4285
Describe the PR
The rule CommentDefaultAccessModifier ignores @VisibleForTesting as stated by the documentation:
This rule ignores by default all cases that have a @VisibleForTesting annotation or any JUnit5 annotation
https://pmd.github.io/pmd/pmd_rules_java_codestyle.html#commentdefaultaccessmodifier
This should also apply to the annotation co.elastic.clients.util.VisibleForTesting from the elastic provided java client, elasticsearch-java,.
Ready?
./mvnw clean verify
passes (checked automatically by github actions)