Issue #7225: bump pmd to 6.18.0

checkstyle · Oct 27, 2019 · c11154c · c11154c
1 parent 5eaada6
commit c11154c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/config/pmd.xml b/config/pmd.xml
@@ -15,6 +15,8 @@
     <exclude name="AccessorMethodGeneration"/>
     <!-- The PMD mistakes the AbstractViolationReporter::log() as a debug log. -->
     <exclude name="GuardLogStatement"/>
+    <!-- too much false-positives from 6.18.0 -->
+    <exclude name="UnusedPrivateMethod"/>
   </rule>
   <rule ref="category/java/bestpractices.xml/AvoidPrintStackTrace">
     <properties>
@@ -75,6 +77,8 @@
     <exclude name="OnlyOneReturn"/>
     <!-- We use CheckstyleCustomShortVariable, to control the length and skip Override methods. -->
     <exclude name="ShortVariable"/>
+    <!-- huge amount of false positives from 6.18.0 -->
+    <exclude name="UnnecessaryFullyQualifiedName"/>
   </rule>
   <rule ref="category/java/codestyle.xml/ClassNamingConventions">
     <properties>

diff --git a/pom.xml b/pom.xml
@@ -196,7 +196,7 @@
     <maven.site.plugin.version>3.8.2</maven.site.plugin.version>
     <maven.spotbugs.plugin.version>3.1.12.2</maven.spotbugs.plugin.version>
     <maven.pmd.plugin.version>3.12.0</maven.pmd.plugin.version>
-    <pmd.version>6.17.0</pmd.version>
+    <pmd.version>6.18.0</pmd.version>
     <maven.jacoco.plugin.version>0.8.5</maven.jacoco.plugin.version>
     <powermock.version>2.0.4</powermock.version>
     <saxon.version>9.9.1-5</saxon.version>