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 <lb...@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>