Apply UnnecessarySemicolonInEnumeration Checkstyle module

This commit also applies it to this project itself and fixes its violations.
spring-io · Mar 23, 2024 · 723779f · 723779f
1 parent 7db7c97
commit 723779f
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 4 deletions.
diff --git a/...ormat-checkstyle/src/main/resources/io/spring/javaformat/checkstyle/spring-checkstyle.xml b/...ormat-checkstyle/src/main/resources/io/spring/javaformat/checkstyle/spring-checkstyle.xml
@@ -113,6 +113,7 @@
  <module name="com.puppycrawl.tools.checkstyle.checks.UpperEllCheck" />
  <module name="com.puppycrawl.tools.checkstyle.checks.ArrayTypeStyleCheck" />
  <module name="com.puppycrawl.tools.checkstyle.checks.OuterTypeFilenameCheck" />
+ <module name="UnnecessarySemicolonInEnumeration"/>
 
  <!-- Modifiers -->
  <module name="com.puppycrawl.tools.checkstyle.checks.modifier.RedundantModifierCheck" />

diff --git a/...ckstyle/src/test/java/io/spring/javaformat/checkstyle/SpringConfigurationLoaderTests.java b/...ckstyle/src/test/java/io/spring/javaformat/checkstyle/SpringConfigurationLoaderTests.java
@@ -48,7 +48,7 @@ public void loadShouldLoadChecks() {
  assertThat(checks).hasSize(5);
  TreeWalker treeWalker = (TreeWalker) checks.toArray()[4];
  Set<?> ordinaryChecks = (Set<?>) Extractors.byName("ordinaryChecks").extract(treeWalker);
- assertThat(ordinaryChecks).hasSize(60);
+ assertThat(ordinaryChecks).hasSize(61);
  }
 
  @Test
@@ -59,7 +59,7 @@ public void loadWithExcludeShouldExcludeChecks() {
  assertThat(checks).hasSize(5);
  TreeWalker treeWalker = (TreeWalker) checks.toArray()[4];
  Set<?> ordinaryChecks = (Set<?>) Extractors.byName("ordinaryChecks").extract(treeWalker);
- assertThat(ordinaryChecks).hasSize(59);
+ assertThat(ordinaryChecks).hasSize(60);
  }
 
  @Test

diff --git a/...matter-eclipse-jdt-jdk17/src/main/java/org/eclipse/jdt/internal/formatter/Preparator.java b/...matter-eclipse-jdt-jdk17/src/main/java/org/eclipse/jdt/internal/formatter/Preparator.java
@@ -54,7 +54,7 @@ enum Phase {
  /**
  * Apply the preparator after wrapping.
  */
- POST_WRAPPING;
+ POST_WRAPPING
 
  }
 

diff --git a/...rmatter-eclipse-jdt-jdk8/src/main/java/org/eclipse/jdt/internal/formatter/Preparator.java b/...rmatter-eclipse-jdt-jdk8/src/main/java/org/eclipse/jdt/internal/formatter/Preparator.java
@@ -54,7 +54,7 @@ enum Phase {
  /**
  * Apply the preparator after wrapping.
  */
- POST_WRAPPING;
+ POST_WRAPPING
 
  }
 

diff --git a/src/checkstyle/checkstyle.xml b/src/checkstyle/checkstyle.xml
@@ -117,6 +117,7 @@
  <module name="com.puppycrawl.tools.checkstyle.checks.UpperEllCheck" />
  <module name="com.puppycrawl.tools.checkstyle.checks.ArrayTypeStyleCheck" />
  <module name="com.puppycrawl.tools.checkstyle.checks.OuterTypeFilenameCheck" />
+ <module name="UnnecessarySemicolonInEnumeration"/>
 
  <!-- Modifiers -->
  <module name="com.puppycrawl.tools.checkstyle.checks.modifier.RedundantModifierCheck" />