Java Autoformatter

.github/workflows/java-autoformat.yml

@@ -0,0 +1,38 @@
+name: Java Code Auto Format
+on: pull_request
+
+jobs:
+  format:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v1
+        with:
+          ref: ${{ github.head_ref }}
+
+      - name: Set up JDK 11
+        uses: actions/setup-java@v1
+        with:
+          java-version: 11
+
+      - name: Grant execute permission for gradlew
+        run: chmod +x gradlew
+
+      - name: Build with Gradle
+        run: ./gradlew build -x test
+
+      - name: Run spotless
+        run: ./gradlew :spotlessApply
+
+      - name: Check for modified files
+        id: git-check
+        run: echo "modified=$(if git diff-index --quiet HEAD --; then echo "false"; else echo "true"; fi)" >> $GITHUB_OUTPUT
+
+      - name: Push changes
+        if: steps.git-check.outputs.modified == 'true'
+        run: |
+          git config --global user.name 'Bram van Heuveln'
+          git config --global user.email '[email protected]'
+          git remote set-url origin https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/${{ github.repository }}
+          git add .
+          git diff --cached --exit-code || git commit -am "Automated Java code formatting changes" && git push

build.gradle

@@ -2,13 +2,43 @@ plugins {
     id 'java'
     id 'edu.sc.seis.launch4j' version '2.5.3'
     id 'kr.motd.sphinx' version '2.10.0'
+    id 'com.diffplug.spotless' version '6.25.0'
 }
 
+
 version '2.0.0'
 
 apply plugin: 'java'
 apply plugin: 'application'
+
+spotless{
+    enforceCheck false
+
+    format 'misc', {
+        // define the files to apply `misc` to
+        target '*.gradle', '*.md', '.gitignore'
+
+        // define the steps to apply to those files
+        trimTrailingWhitespace()
+        indentWithSpaces() // or spaces. Takes an integer argument if you don't like 4
+        endWithNewline()
+    }
+
+    java{
+        // Use the default importOrder configuration
+        importOrder()
+
+        // Cleanthat will refactor your code, but it may break your style: apply it before your formatter
+        cleanthat()
+
+        googleJavaFormat('1.19.2').aosp()
+
+        formatAnnotations()
+    }
+}
+
 apply plugin: 'checkstyle'
+
 mainClassName = 'edu.rpi.legup.Legup'
 
 sourceCompatibility = 11

config/checkstyle/checkstyle.xml

@@ -7,12 +7,6 @@
 <module name="Checker">
     <module name="TreeWalker">
         <property name="tabWidth" value="4"/>
-        <module name="LeftCurly"/>
-        <module name="RightCurly">
-            <property name="option" value="alone"/>
-            <property name="tokens"
-                      value="LITERAL_IF, LITERAL_ELSE, LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_WHILE, LITERAL_DO, LITERAL_FOR, METHOD_DEF"/>
-        </module>
         <module name="NeedBraces">
             <property name="allowSingleLineStatement" value="true"/>
         </module>

src/main/java/edu/rpi/legup/Legup.java

@@ -15,4 +15,4 @@ public static void main(String[] args) {
         GameBoardFacade.getInstance();
         GameBoardFacade.setupConfig();
     }
-}
+}

src/main/java/edu/rpi/legup/ai/Solver.java

@@ -1,4 +1,3 @@
 package edu.rpi.legup.ai;
 
-public class Solver {
-}
+public class Solver {}

src/main/java/edu/rpi/legup/app/Config.java

@@ -2,21 +2,18 @@
 
 import java.io.*;
 import java.util.*;
-
 import javax.xml.parsers.DocumentBuilder;
 import javax.xml.parsers.DocumentBuilderFactory;
 import javax.xml.parsers.ParserConfigurationException;
-
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.w3c.dom.NodeList;
 import org.xml.sax.SAXException;
 
 public class Config {
-    private final static Logger Logger = LogManager.getLogger(Config.class.getName());
+    private static final Logger Logger = LogManager.getLogger(Config.class.getName());
 
     private Map<String, String> puzzles;
     private Map<String, Boolean> fileCreationDisabledStatuses;
@@ -43,8 +40,8 @@ public List<String> getPuzzleClassNames() {
     }
 
     /**
-     * Returns a list of the names of the puzzles which can have puzzles created and edited
-     * within the proof editor.
+     * Returns a list of the names of the puzzles which can have puzzles created and edited within
+     * the proof editor.
      *
      * @return the aforementioned list of Strings
      */
@@ -59,8 +56,8 @@ public List<String> getFileCreationEnabledPuzzles() {
     }
 
     /**
-     * Converts the class name of the puzzles to their display names. Some examples of the conversion:
-     * convertClassNameToDisplayName("TreeTent") will return "Tree Tent"
+     * Converts the class name of the puzzles to their display names. Some examples of the
+     * conversion: convertClassNameToDisplayName("TreeTent") will return "Tree Tent"
      * convertClassNameToDisplayName("Nurikabe") will return "Nurikabe"
      *
      * @param className the name of the class
@@ -140,15 +137,16 @@ private void loadConfig(InputStream stream) throws InvalidConfigException {
                 Element puzzle = (Element) puzzleNodes.item(i);
                 String name = puzzle.getAttribute("name");
                 String className = puzzle.getAttribute("qualifiedClassName");
-                boolean status = Boolean.parseBoolean(puzzle.getAttribute("fileCreationDisabled").toLowerCase());
+                boolean status =
+                        Boolean.parseBoolean(
+                                puzzle.getAttribute("fileCreationDisabled").toLowerCase());
                 Logger.debug("Class Name: " + className);
                 this.puzzles.put(name, className);
                 this.fileCreationDisabledStatuses.put(name, Boolean.valueOf(status));
             }
 
-        }
-        catch (ParserConfigurationException | SAXException | IOException e) {
+        } catch (ParserConfigurationException | SAXException | IOException e) {
             throw new InvalidConfigException(e.getMessage());
         }
     }
-}
+}