Review round 1

java-checks/src/main/java/org/sonar/java/checks/helpers/AnnotationsHelper.java

@@ -28,7 +28,9 @@ public static boolean hasUnknownAnnotation(SymbolMetadata symbolMetadata) {
     return symbolMetadata.annotations().stream().anyMatch(annotation -> annotation.symbol().isUnknown());
   }
 
-  /// Returns the `name` part of a `fully.qualified.name`, that is, the part after the last dot.
+  /**
+   * Returns the `name` part of a `fully.qualified.name`, that is, the part after the last dot.
+   */
   public static String annotationTypeIdentifier(String fullyQualified) {
     return fullyQualified.substring(fullyQualified.lastIndexOf('.') + 1);
   }

java-checks/src/main/java/org/sonar/java/checks/tests/IgnoredTestsCheck.java

@@ -64,7 +64,7 @@ public void visitNode(Tree tree) {
             "Either add an explanation about why this test is skipped or remove the \"@%s\" annotation.",
             shortName
           );
-          JavaFileScannerContext.Location secondaryLocation =
+          var secondaryLocation =
             new JavaFileScannerContext.Location(String.format("@%s annotation skips the test", shortName), annotationTree);
           context.reportIssue(this, methodTree.simpleName(), message, Collections.singletonList(secondaryLocation), null);
         });
@@ -91,7 +91,9 @@ public void visitNode(Tree tree) {
     }
   }
 
-  /// If a test method is silently ignored, returns the annotation that causes this behavior.
+  /**
+   * If a test method is silently ignored, returns the annotation that causes this behavior.
+   */
   private static Optional<AnnotationTree> getSilentlyIgnoredAnnotation(SymbolMetadata symbolMetadata, String fullyQualifiedName) {
     // This code duplicates the behavior of SymbolMetadata.valuesForAnnotation but checks for broken semantics
     for (SymbolMetadata.AnnotationInstance annotation : symbolMetadata.annotations()) {