This is an automated email from the ASF dual-hosted git repository.

dkulp pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cxf-build-utils.git


The following commit(s) were added to refs/heads/master by this push:
     new 3b54d4f  Update to latest PMD rules
3b54d4f is described below

commit 3b54d4fa0ad5573006e887f1446a246dfca44dd5
Author: Daniel Kulp <d...@kulp.com>
AuthorDate: Thu Apr 14 13:39:32 2022 -0400

    Update to latest PMD rules
---
 buildtools/src/main/resources/cxf-pmd-ruleset.xml | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/buildtools/src/main/resources/cxf-pmd-ruleset.xml 
b/buildtools/src/main/resources/cxf-pmd-ruleset.xml
index f1afcc3..a3e07a9 100644
--- a/buildtools/src/main/resources/cxf-pmd-ruleset.xml
+++ b/buildtools/src/main/resources/cxf-pmd-ruleset.xml
@@ -53,18 +53,23 @@
         <exclude name="JUnitUseExpected"/>
         <exclude name="JUnit4TestShouldUseAfterAnnotation"/>
         <exclude name="JUnit4TestShouldUseBeforeAnnotation" />
-        <exclude name="JUnit4TestShouldUseTestAnnotation"/>
+        <exclude name="JUnit4TestShouldUseTestAnnotation" />
+        <exclude name="JUnit5TestShouldBePackagePrivate" />
         <exclude name="MethodReturnsInternalArray" />
         <exclude name="MissingOverride" />
+        <exclude name="LiteralsFirstInComparisons" />
         <exclude name="OneDeclarationPerLine" />
         <exclude name="PreserveStackTrace" />
         <exclude name="ReplaceEnumerationWithIterator" />
+        <exclude name="SimplifiableTestAssertion" />
         <exclude name="SwitchStmtsShouldHaveDefault" />
         <exclude name="SystemPrintln" />
+        <exclude name="UnusedAssignment" />
         <exclude name="UnusedFormalParameter" />
         <exclude name="UnusedPrivateMethod" />
         <exclude name="UseAssertEqualsInsteadOfAssertTrue" />
         <exclude name="UseAssertSameInsteadOfAssertTrue" />
+        <exclude name="UseTryWithResources" />
         <exclude name="UseVarargs" />
     </rule>
 
@@ -75,6 +80,7 @@
         <exclude name="AvoidThrowingNullPointerException" />
         <exclude name="AvoidThrowingRawExceptionTypes" />
         <exclude name="AvoidUncheckedExceptionsInSignatures" />
+        <exclude name="CognitiveComplexity" />
         <exclude name="CouplingBetweenObjects" />
         <exclude name="CyclomaticComplexity" />
         <exclude name="DataClass" />
@@ -87,6 +93,7 @@
         <exclude name="ImmutableField" />
         <exclude name="LawOfDemeter" />
         <exclude name="LoosePackageCoupling" />
+        <exclude name="MutableStaticState" />
         <exclude name="NcssCount" />
         <exclude name="NPathComplexity" />
         <exclude name="SignatureDeclareThrowsException" />
@@ -106,6 +113,7 @@
         <exclude name="AvoidAccessibilityAlteration" />
         <exclude name="AvoidCatchingNPE" />
         <exclude name="AvoidCatchingThrowable" />
+        <exclude name="AvoidDecimalLiteralsInBigDecimalConstructor" />
         <exclude name="AvoidDuplicateLiterals" />
         <exclude name="AvoidFieldNameMatchingMethodName" />
         <exclude name="AvoidFieldNameMatchingTypeName" />
@@ -117,18 +125,18 @@
         <exclude name="ConstructorCallsOverridableMethod" />
         <exclude name="DataflowAnomalyAnalysis" />
         <exclude name="DoNotCallGarbageCollectionExplicitly" />
-        <exclude name="DoNotCallSystemExit" />
         <exclude name="DoNotThrowExceptionInFinally" />
         <exclude name="EmptyCatchBlock" />
         <exclude name="EmptyIfStmt" />
         <exclude name="EmptySynchronizedBlock" />
         <exclude name="EmptyWhileStmt" />
-        <exclude name="MissingBreakInSwitch" />
+        <exclude name="ImplicitSwitchFallThrough" />
         <exclude name="MissingSerialVersionUID" />
         <exclude name="MissingStaticMethodInNonInstantiatableClass" />
         <exclude name="MoreThanOneLogger" />
         <exclude name="NullAssignment" />
         <exclude name="ReturnEmptyArrayRatherThanNull" />
+        <exclude name="ReturnEmptyCollectionRatherThanNull" />
         <exclude name="SimpleDateFormatNeedsLocale" />
         <exclude name="SingletonClassReturningNewInstance" />
         <exclude name="SingleMethodSingleton" />

Reply via email to