Rename RemoveWildcardImports to ForbidWildcardImports in maven.

Author: nedtwigg

plugin-maven/README.md

@@ -230,7 +230,7 @@ any other maven phase (i.e. compile) then it can be configured as below;
     </importOrder>
 
     <removeUnusedImports /> <!-- self-explanatory -->
-    <removeWildcardImports /> <!-- drop any import ending with '*' -->
+    <forbidWildcardImports /> <!-- yell if any import ends with '*' -->
 
     <formatAnnotations />  <!-- fixes formatting of type annotations, see below -->
 
@@ -249,10 +249,10 @@ any other maven phase (i.e. compile) then it can be configured as below;
 </removeUnusedImports>
 ```
 
-### removeWildcardImports
+### forbidWildcardImports
 
 ```xml
-<removeWildcardImports/>
+<forbidWildcardImports/>
 ```
 
 ### google-java-format
@@ -1950,7 +1950,7 @@ Sometimes Spotless will encounter lint errors that can't be auto-fixed. For exam
 
 ```
 [ERROR] Unable to format file src/main/java/com/example/App.java
-[ERROR] Step 'removeWildcardImports' found problem in 'App.java':
+[ERROR] Step 'forbidWildcardImports' found problem in 'App.java':
 [ERROR]   Do not use wildcard imports
 ```
 
@@ -1963,12 +1963,12 @@ To suppress these lints, you can use the `<lintSuppressions>` configuration:
   <version>${spotless.version}</version>
   <configuration>
     <java>
-      <removeWildcardImports/>
+      <forbidWildcardImports/>
     </java>
     <lintSuppressions>
       <lintSuppression>
         <path>src/main/java/com/example/App.java</path>
-        <step>removeWildcardImports</step>
+        <step>forbidWildcardImports</step>
         <shortCode>*</shortCode>
       </lintSuppression>
     </lintSuppressions>
@@ -1988,13 +1988,13 @@ You can suppress multiple patterns:
   <!-- Suppress all wildcard import errors in legacy code -->
   <lintSuppression>
     <path>src/main/java/com/example/legacy/*</path>
-    <step>removeWildcardImports</step>
+    <step>forbidWildcardImports</step>
     <shortCode>*</shortCode>
   </lintSuppression>
   <!-- Suppress all errors from a specific step -->
   <lintSuppression>
     <path>*</path>
-    <step>removeWildcardImports</step>
+    <step>forbidWildcardImports</step>
     <shortCode>*</shortCode>
   </lintSuppression>
 </lintSuppressions>

plugin-maven/src/main/java/com/diffplug/spotless/maven/java/RemoveWildcardImports.java renamed to plugin-maven/src/main/java/com/diffplug/spotless/maven/java/ForbidWildcardImports.java

@@ -20,7 +20,7 @@
 import com.diffplug.spotless.maven.FormatterStepConfig;
 import com.diffplug.spotless.maven.FormatterStepFactory;
 
-public class RemoveWildcardImports implements FormatterStepFactory {
+public class ForbidWildcardImports implements FormatterStepFactory {
 	@Override
 	public FormatterStep newFormatterStep(FormatterStepConfig config) {
 		return RemoveWildcardImportsStep.create();

plugin-maven/src/main/java/com/diffplug/spotless/maven/java/Java.java

@@ -76,8 +76,8 @@ public void addRemoveUnusedImports(RemoveUnusedImports removeUnusedImports) {
 		addStepFactory(removeUnusedImports);
 	}
 
-	public void addRemoveWildcardImports(RemoveWildcardImports removeWildcardImports) {
-		addStepFactory(removeWildcardImports);
+	public void addRemoveWildcardImports(ForbidWildcardImports forbidWildcardImports) {
+		addStepFactory(forbidWildcardImports);
 	}
 
 	public void addFormatAnnotations(FormatAnnotations formatAnnotations) {

plugin-maven/src/test/java/com/diffplug/spotless/maven/java/ForbidWildcardImportsStepTest.java

@@ -0,0 +1,33 @@
+/*
+ * Copyright 2025 DiffPlug
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.diffplug.spotless.maven.java;
+
+import org.junit.jupiter.api.Test;
+
+import com.diffplug.spotless.maven.MavenIntegrationHarness;
+
+class ForbidWildcardImportsStepTest extends MavenIntegrationHarness {
+
+	@Test
+	void testRemoveWildcardImports() throws Exception {
+		writePomWithJavaSteps("<removeWildcardImports/>");
+
+		String path = "src/main/java/test.java";
+		setFile(path).toResource("java/removewildcardimports/JavaCodeWildcardsUnformatted.test");
+		mavenRunner().withArguments("spotless:apply").runNoError();
+		assertFile(path).sameAsResource("java/removewildcardimports/JavaCodeWildcardsFormatted.test");
+	}
+}