[camel-k-runtime] 05/10: chore: update pmd rules

[lburgazzoli]

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

lburgazzoli pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k-runtime.git

commit ac77426258ecd3754f700deb6321014476a04ea9
Author: Luca Burgazzoli <lburgazz...@gmail.com>
AuthorDate: Wed Aug 17 17:23:57 2022 +0200

    chore: update pmd rules
---
 support/pmd-ruleset.xml | 22 +++++-----------------
 1 file changed, 5 insertions(+), 17 deletions(-)

diff --git a/support/pmd-ruleset.xml b/support/pmd-ruleset.xml
index 9c666c8e..c4a68463 100644
--- a/support/pmd-ruleset.xml
+++ b/support/pmd-ruleset.xml
@@ -27,8 +27,10 @@
     
<exclude-pattern>.*/org/apache/camel/k/loader/yaml/support/.*</exclude-pattern>
     
<exclude-pattern>.*/org/apache/camel/k/loader/yaml/parser/.*</exclude-pattern>
 
+
+    <rule ref="category/java/bestpractices.xml/SimplifiableTestAssertion" />
+    <rule ref="category/java/bestpractices.xml/PrimitiveWrapperInstantiation" 
/>
     <rule ref="category/java/bestpractices.xml/AvoidUsingHardCodedIP"/>
-    <rule ref="category/java/bestpractices.xml/UnusedImports"/>
     <rule ref="category/java/bestpractices.xml/UnusedFormalParameter"/>
     <rule ref="category/java/bestpractices.xml/UnusedLocalVariable"/>
     <rule ref="category/java/bestpractices.xml/UnusedPrivateField"/>
@@ -37,8 +39,8 @@
     <rule ref="category/java/bestpractices.xml/UseCollectionIsEmpty"/>
     <rule ref="category/java/bestpractices.xml/PreserveStackTrace"/>
 
-    <rule ref="category/java/codestyle.xml/DontImportJavaLang"/>
-    <rule ref="category/java/codestyle.xml/DuplicateImports"/>
+    <rule ref="category/java/codestyle.xml/UnnecessaryImport" />
+    <rule ref="category/java/codestyle.xml/EmptyControlStatement" />
     <rule ref="category/java/codestyle.xml/ExtendsObject"/>
     <rule ref="category/java/codestyle.xml/ForLoopShouldBeWhileLoop"/>
     <rule ref="category/java/codestyle.xml/UnnecessaryFullyQualifiedName"/>
@@ -54,9 +56,6 @@
     <rule ref="category/java/design.xml/SimplifiedTernary">
         <priority>4</priority>
     </rule>
-    <rule ref="category/java/design.xml/SimplifyBooleanAssertion">
-        <priority>4</priority>
-    </rule>
     <rule ref="category/java/design.xml/CollapsibleIfStatements">
         <priority>4</priority>
     </rule>
@@ -79,16 +78,6 @@
         </properties>
     </rule>
 
-    <rule ref="category/java/errorprone.xml/EmptyFinallyBlock"/>
-    <rule ref="category/java/errorprone.xml/EmptyIfStmt"/>
-    <rule ref="category/java/errorprone.xml/EmptyInitializer"/>
-    <rule ref="category/java/errorprone.xml/EmptyStatementBlock"/>
-    <rule ref="category/java/errorprone.xml/EmptyStatementNotInLoop"/>
-    <rule ref="category/java/errorprone.xml/EmptySwitchStatements"/>
-    <rule ref="category/java/errorprone.xml/EmptySynchronizedBlock"/>
-    <rule ref="category/java/errorprone.xml/EmptyTryBlock"/>
-    <rule ref="category/java/errorprone.xml/EmptyWhileStmt"/>
-    <rule ref="category/java/errorprone.xml/ImportFromSamePackage"/>
     <rule ref="category/java/errorprone.xml/JumbledIncrementer"/>
     <rule ref="category/java/errorprone.xml/MisplacedNullCheck"/>
     <rule ref="category/java/errorprone.xml/OverrideBothEqualsAndHashcode"/>
@@ -104,5 +93,4 @@
     <rule ref="category/java/multithreading.xml/DontCallThreadRun"/>
 
     <rule ref="category/java/performance.xml/BigIntegerInstantiation"/>
-    <rule ref="category/java/performance.xml/BooleanInstantiation"/>
 </ruleset>