vishesh92 commented on code in PR #13168: URL: https://github.com/apache/cloudstack/pull/13168#discussion_r3266050739
########## tools/checkstyle/src/main/resources/cloud-pmd.xml: ########## @@ -23,59 +23,88 @@ xmlns="http://pmd.sourceforge.net/ruleset/2.0.0";> <description> - Ruleset that brings all the rulesets we want from the pmd jar, because - the maven-pmd-plugin doesn't find them otherwise. This is also the place - to add our own future rulesets, if any. + CloudStack PMD ruleset. Enables all PMD 7 categories with + project-specific exclusions for rules that are too noisy + or incompatible with CloudStack conventions. </description> - <rule ref="category/java/basic.xml"/> - <rule ref="category/java/braces.xml"/> - <rule ref="category/java/clone.xml"/> - <rule ref="category/java/codesize.xml"/> - <rule ref="category/java/comments.xml"> - <!-- We shouldn't limit the number of lines in the header of a class --> - <exclude name="CommentSize"/> - <exclude name="CommentRequired" /> + <rule ref="category/java/bestpractices.xml"> + <exclude name="AbstractClassWithoutAbstractMethod" /> + <exclude name="GuardLogStatement" /> + <!-- <exclude name="LiteralsFirstInComparisons" /> --> Review Comment: I discussed this with @DaanHoogland offline. And he suggested to comment these for now and enable as per requirement going forward. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
