Merge branch 'main' into DDC_JDK11

Author: dedece35

.github/dependabot.yml

@@ -14,3 +14,5 @@ updates:
       - dependency-name: "org.sonarsource.java:sonar-java-plugin"
       # Ignore all versions : cf pom.xml comments
       - dependency-name: "com.mycila:license-maven-plugin"
+      - dependency-name: "org.springframework.data:spring-data-commons"
+        update-types: ["version-update:semver-patch", "version-update:semver-minor"]

pom.xml

@@ -219,12 +219,12 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
-                <version>3.13.0</version>
+                <version>3.14.0</version>
             </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
-                <version>3.5.2</version>
+                <version>3.5.3</version>
             </plugin>
             <plugin>
                 <groupId>org.jacoco</groupId>
@@ -357,13 +357,13 @@
                                 <artifactItem>
                                     <groupId>org.springframework</groupId>
                                     <artifactId>spring-webmvc</artifactId>
-                                    <version>5.2.3.RELEASE</version>
+                                    <version>6.2.10</version>
                                     <type>jar</type>
                                 </artifactItem>
                                 <artifactItem>
                                     <groupId>org.springframework</groupId>
                                     <artifactId>spring-web</artifactId>
-                                    <version>5.2.3.RELEASE</version>
+                                    <version>6.2.10</version>
                                     <type>jar</type>
                                 </artifactItem>
                                 <artifactItem>
@@ -381,7 +381,11 @@
                                 <artifactItem>
                                     <groupId>org.springframework.data</groupId>
                                     <artifactId>spring-data-commons</artifactId>
-                                    <version>2.2.4.RELEASE</version>
+                                    <!-- <version>2.2.4.RELEASE</version> -->
+                                    <version>2.7.18</version>
+                                    <!-- KO with version 3.x -->
+                                    <!-- <version>3.0.0</version> -->
+                                    <!-- <version>3.5.3</version> -->
                                     <type>jar</type>
                                 </artifactItem>
                             </artifactItems>
@@ -465,7 +469,7 @@
                 <!-- Execute integration tests -->
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-failsafe-plugin</artifactId>
-                <version>3.5.2</version>
+                <version>3.5.3</version>
                 <executions>
                     <execution>
                         <goals>

src/main/java/org/greencodeinitiative/creedengo/java/checks/MakeNonReassignedVariablesConstants.java

@@ -25,12 +25,13 @@ public List<Kind> nodesToVisit() {
     @Override
     public void visitNode(@Nonnull Tree tree) {
         VariableTree variableTree = (VariableTree) tree;
-        LOGGER.debug("Variable > " + getVariableNameForLogger(variableTree));
-        LOGGER.debug("   => isNotFinalAndNotStatic(variableTree) = " + isNotFinalAndNotStatic(variableTree));
-        LOGGER.debug("   => usages = " + variableTree.symbol().usages().size());
-        LOGGER.debug("   => isNotReassigned = " + isNotReassigned(variableTree));
-        LOGGER.debug("   => isPassedAsNonFinalParameter = " + isPassedAsNonFinalParameter(variableTree));
-
+        if (LOGGER.isDebugEnabled()) {
+            LOGGER.debug("Variable > {}", getVariableNameForLogger(variableTree));
+            LOGGER.debug("   => isNotFinalAndNotStatic(variableTree) = {}", isNotFinalAndNotStatic(variableTree));
+            LOGGER.debug("   => usages = {}", variableTree.symbol().usages().size());
+            LOGGER.debug("   => isNotReassigned = {}", isNotReassigned(variableTree));
+            LOGGER.debug("   => isPassedAsNonFinalParameter = {}", isPassedAsNonFinalParameter(variableTree));
+        }
         if (isNotFinalAndNotStatic(variableTree) && isNotReassigned(variableTree)) {
             reportIssue(tree, MESSAGE_RULE);
         } else {