Added missing user warnings for deprecated methods.

diffplug · Aug 1, 2017 · 6a3f8ad · 6a3f8ad
1 parent bfb47a8
commit 6a3f8ad
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/GroovyExtension.java b/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/GroovyExtension.java
@@ -29,6 +29,7 @@
 import org.gradle.api.tasks.GroovySourceSet;
 import org.gradle.api.tasks.SourceSet;
 
+import com.diffplug.common.base.StringPrinter;
 import com.diffplug.spotless.FormatterStep;
 import com.diffplug.spotless.SerializableFileFilter;
 import com.diffplug.spotless.extra.groovy.GrEclipseFormatterStep;
@@ -66,6 +67,11 @@ public void licenseHeaderFile(Object licenseHeaderFile) {
 	 * {@link GroovyExtension#importOrder(String...)}.*/
 	@Deprecated
 	public void importOrder(List<String> importOrder) {
+		getProject().getLogger().warn(
+				StringPrinter.buildStringFromLines(
+						"'importOrder([x, y, z])' is deprecated.",
+						"Use 'importOrder x, y, z' instead.",
+						"For details see https://github.com/diffplug/spotless/tree/master/plugin-gradle#applying-to-java-source"));
 		addStep(ImportOrderStep.createFromOrder(importOrder));
 	}
 

diff --git a/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/JavaExtension.java b/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/JavaExtension.java
@@ -58,6 +58,11 @@ public void licenseHeaderFile(Object licenseHeaderFile) {
 	 * {@link JavaExtension#importOrder(String...)}.*/
 	@Deprecated
 	public void importOrder(List<String> importOrder) {
+		getProject().getLogger().warn(
+				StringPrinter.buildStringFromLines(
+						"'importOrder([x, y, z])' is deprecated.",
+						"Use 'importOrder x, y, z' instead.",
+						"For details see https://github.com/diffplug/spotless/tree/master/plugin-gradle#applying-to-java-source"));
 		addStep(ImportOrderStep.createFromOrder(importOrder));
 	}