exceptionfactory commented on PR #9581: URL: https://github.com/apache/nifi/pull/9581#issuecomment-2546115950
@dan-s1 For reference, here is the default ruleset configuration for the Maven PMD plugin: https://github.com/apache/maven-pmd-plugin/blob/master/src/main/resources/rulesets/java/maven-pmd-plugin-default.xml Most of the removed rulesets relate to unused elements and unnecessary syntax. Removing unused values involves code changes in cases that may impact behavior, so that was the reason for removing some of those rules. Removing unnecessary syntax should be safer in general, but touches more files. Now that the PMD check is part of the main branch, we can certainly consider adding some of these rulesets on a case-by-case basis. -- 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]
