build: Add simple spotless checks for import order and annotations

Author: zml2008

.checkstyle/checkstyle.xml

@@ -271,6 +271,8 @@
         <module name="CustomImportOrder">
             <property name="sortImportsInGroupAlphabetically" value="true"/>
             <property name="separateLineBetweenGroups" value="true"/>
+            <property name="standardPackageRegExp" value="^java\."/>
+            <property name="specialImportsRegExp" value="^javax\."/>
             <!-- Sponge - Custom import order -->
             <property name="customImportOrderRules" value="STATIC###THIRD_PARTY_PACKAGE###STANDARD_JAVA_PACKAGE###SPECIAL_IMPORTS"/>
             <property name="tokens" value="IMPORT, STATIC_IMPORT, PACKAGE_DEF"/>

.editorconfig

@@ -0,0 +1,15 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+indent_size = 4
+indent_style = space
+insert_final_newline = true
+max_line_length = 150
+tab_width = 4
+
+[*.java]
+ij_java_imports_layout = $*,|,*,|,java.**,|,javax.**
+ij_java_insert_inner_class_imports = false
+ij_java_layout_static_imports_separately = true

build.gradle.kts

@@ -244,7 +244,7 @@ spongeConvention {
 
 indra {
     val checkstyleVersion: String by project
-    
+
     javaVersions {
         testWith(8, 11, 17)
     }
@@ -263,6 +263,22 @@ indraCrossdoc {
     baseUrl(providers.gradleProperty("javadocPublishRoot"))
 }
 
+spotless {
+    java {
+        endWithNewline()
+        indentWithSpaces(4)
+        trimTrailingWhitespace()
+        toggleOffOn("@formatter:off", "@formatter:on")
+        formatAnnotations()
+        importOrderFile(rootProject.file("extra/eclipse/sponge_eclipse.importorder"))
+    }
+    kotlinGradle {
+        endWithNewline()
+        indentWithSpaces(4)
+        trimTrailingWhitespace()
+    }
+}
+
 val sortClasses = listOf(
         "org.spongepowered.api.advancement.AdvancementTypes",
         "org.spongepowered.api.advancement.criteria.trigger.Triggers",
@@ -389,4 +405,4 @@ val sortClasses = listOf(
         "org.spongepowered.api.world.schematic.PaletteTypes",
         "org.spongepowered.api.world.teleport.TeleportHelperFilters",
         "org.spongepowered.api.world.weather.WeatherTypes"
-)
+)

src/ap/java/org/spongepowered/plugin/processor/ListenerParameterAnnotation.java

@@ -35,6 +35,7 @@
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Set;
+
 import javax.lang.model.element.AnnotationValue;
 import javax.lang.model.element.Element;
 import javax.lang.model.element.ElementKind;

src/ap/java/org/spongepowered/plugin/processor/ListenerProcessor.java

@@ -37,6 +37,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+
 import javax.annotation.processing.AbstractProcessor;
 import javax.annotation.processing.RoundEnvironment;
 import javax.annotation.processing.SupportedAnnotationTypes;

src/ap/java/org/spongepowered/plugin/processor/ParameterContext.java

@@ -28,6 +28,7 @@
 
 import java.util.Objects;
 import java.util.Optional;
+
 import javax.annotation.processing.ProcessingEnvironment;
 import javax.lang.model.element.AnnotationMirror;
 import javax.lang.model.element.AnnotationValue;

src/ap/java/org/spongepowered/plugin/processor/ProcessorUtils.java

@@ -28,6 +28,7 @@
 
 import java.util.Collection;
 import java.util.List;
+
 import javax.lang.model.element.Element;
 import javax.lang.model.element.ElementKind;
 import javax.lang.model.element.TypeElement;

src/main/java/org/spongepowered/api/ResourceKey.java

@@ -74,8 +74,7 @@ public interface ResourceKey extends Key {
      * @param value The value
      * @return A new resource key
      */
-    @NonNull
-    static ResourceKey brigadier(final String value) {
+    static @NonNull ResourceKey brigadier(final String value) {
         return ResourceKey.of(ResourceKey.BRIGADIER_NAMESPACE, value);
     }
 
@@ -85,8 +84,7 @@ static ResourceKey brigadier(final String value) {
      * @param value The value
      * @return A new resource key
      */
-    @NonNull
-    static ResourceKey minecraft(final String value) {
+    static @NonNull ResourceKey minecraft(final String value) {
         return ResourceKey.of(ResourceKey.MINECRAFT_NAMESPACE, value);
     }
 
@@ -96,8 +94,7 @@ static ResourceKey minecraft(final String value) {
      * @param value The value
      * @return A new resource key
      */
-    @NonNull
-    static ResourceKey sponge(final String value) {
+    static @NonNull ResourceKey sponge(final String value) {
         return ResourceKey.of(ResourceKey.SPONGE_NAMESPACE, value);
     }
 
@@ -166,7 +163,7 @@ default String asString() {
     }
 
     @Override
-    default int compareTo(Key o) {
+    default int compareTo(final Key o) {
         return Key.super.compareTo(o);
     }
 
@@ -195,7 +192,7 @@ interface Builder extends org.spongepowered.api.util.Builder<ResourceKey, Builde
          * @param container The plugin container to fetch from
          * @return This builder, for chaining
          */
-        default Builder namespace(PluginContainer container) {
+        default Builder namespace(final PluginContainer container) {
             return this.namespace(container.metadata().id());
         }
 

src/main/java/org/spongepowered/api/block/entity/BlockEntityType.java

@@ -47,4 +47,3 @@ public interface BlockEntityType extends DefaultedRegistryValue {
      */
     boolean isValidBlock(BlockState block);
 }
-

src/main/java/org/spongepowered/api/command/parameter/CommonParameters.java

@@ -39,7 +39,7 @@
  * used when creating commands and retrieving elements. A general workflow for
  * defining a command with a {@link ServerPlayer} parameter may look like this:
  * </p>
- * 
+ *
  * <pre> {@code
  * final Parameter.Value<ServerPlayer> parameter = Parameter.player().setKey("player").build();
  * final Command.Parameterized builder = Command.builder()
@@ -51,9 +51,9 @@
  * // registration happens here.
  * }</pre>
  *
- * <p>While this is a totally valid approach, there are two particular 
+ * <p>While this is a totally valid approach, there are two particular
  * considerations that can be made:</p>
- * 
+ *
  * <ul>
  *     <li>Sponge provided {@link ValueParameter}s and, by extension,
  *     {@link Parameter.Value}s are stateless and so can be reused by multiple
@@ -67,7 +67,7 @@
  * </ul>
  *
  * <p>Given these two considerations, it generally makes sense to store one
- * {@link Parameter.Value} for a given {@link ValueParameter} and 
+ * {@link Parameter.Value} for a given {@link ValueParameter} and
  * {@link Parameter.Key} combination. Further, it is clear that plugins will
  * tend to use similar, if not the same, parameter/key combinations.</p>
  *