Update for checkstyle

pom.xml

@@ -71,7 +71,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
-        <version>3.3</version>
+        <version>3.8.0</version>
         <configuration>
           <source>1.8</source>
           <target>1.8</target>
@@ -132,7 +132,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
-        <version>3.1.1</version>
+        <version>3.2.0</version>
         <dependencies>
           <dependency>
             <groupId>com.puppycrawl.tools</groupId>

src/main/java/com/trickl/assertj/core/api/json/AbstractJsonAssert.java

@@ -1,11 +1,9 @@
 package com.trickl.assertj.core.api.json;
 
-import static org.skyscreamer.jsonassert.JSONCompareMode.NON_EXTENSIBLE;
 
 import com.trickl.assertj.core.internal.Json;
 import com.trickl.assertj.core.util.diff.PostComparisonAction;
 import com.trickl.assertj.core.util.diff.WriteOnFailureAction;
-
 import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
@@ -27,17 +25,17 @@ public abstract class AbstractJsonAssert<S extends AbstractJsonAssert<S>>
     extends AbstractAssert<S, JsonContainer> {
 
   @VisibleForTesting Json json = Json.instance();
-  
+
   private boolean extensible = false;
-  
+
   private boolean strictOrder = true;
-  
+
   private boolean writeActualOnFailure = false;
-  
+
   private Path actualPathOnFailure = null;
-  
+
   private boolean writeExpectedOnFailure = false;
-  
+
   private Path expectedPathOnFailure = null;
 
   public AbstractJsonAssert(JsonContainer actual, Class<?> selfType) {
@@ -59,25 +57,26 @@ public S isSameJsonAs(String expected) {
 
   /**
    * Check if the supplied JSON is the same as expected.
+   *
    * @param expected The expected JSON
    * @return the assertion
    */
   public S isSameJsonAs(JsonContainer expected) {
-    JSONComparator comparator = new DefaultComparator(
-        JSONCompareMode.STRICT
-            .withExtensible(extensible)
-            .withStrictOrdering(strictOrder));
-
+    JSONComparator comparator =
+        new DefaultComparator(
+            JSONCompareMode.STRICT.withExtensible(extensible).withStrictOrdering(strictOrder));
+
     PostComparisonAction postComparisonAction = null;
     if (writeActualOnFailure || writeExpectedOnFailure) {
-      postComparisonAction = new WriteOnFailureAction(
-          info,
-          writeActualOnFailure,
-          actualPathOnFailure,
-          writeExpectedOnFailure,
-          expectedPathOnFailure);
+      postComparisonAction =
+          new WriteOnFailureAction(
+              info,
+              writeActualOnFailure,
+              actualPathOnFailure,
+              writeExpectedOnFailure,
+              expectedPathOnFailure);
     }
-    
+
     json.assertSameJsonAs(info, actual, expected, comparator, postComparisonAction);
     return myself;
   }
@@ -91,17 +90,19 @@ public S allowingAnyArrayOrdering() {
     strictOrder = false;
     return myself;
   }
-  
+
   /**
    * Write the actual output to a temporary file on failure.
+   *
    * @return Updated assertion object
    */
   public S writeActualToFileOnFailure() {
     return writeActualToFileOnFailure(null);
   }
-  
+
   /**
    * Write the actual output to a file on failure.
+   *
    * @param path The file path, if null, a temporary file is used
    * @return Updated assertion object
    */
@@ -110,23 +111,25 @@ public S writeActualToFileOnFailure(Path path) {
     actualPathOnFailure = path;
     return myself;
   }
-  
+
   /**
    * Write the expected output to a temporary file on failure.
+   *
    * @return Updated assertion object
    */
   public S writeExpectedToFileOnFailure() {
     return writeExpectedToFileOnFailure(null);
   }
-  
+
   /**
    * Write the expected output to a file on failure.
+   *
    * @param path The file path, if null, a temporary file is used
    * @return Updated assertion object
    */
   public S writeExpectedToFileOnFailure(Path path) {
     writeExpectedOnFailure = true;
     expectedPathOnFailure = path;
     return myself;
-  }   
+  }
 }

src/main/java/com/trickl/assertj/core/internal/Json.java

@@ -2,7 +2,6 @@
 
 import com.trickl.assertj.core.api.json.JsonContainer;
 import com.trickl.assertj.core.util.diff.PostComparisonAction;
-
 import java.io.ByteArrayInputStream;
 import java.io.File;
 import java.io.IOException;
@@ -34,7 +33,8 @@ public static Json instance() {
   @VisibleForTesting Failures failures = Failures.instance();
 
   @VisibleForTesting
-  Json() {}
+  Json() {
+  }
 
   /**
    * Asserts that the given JSON structures have same content.

src/main/java/com/trickl/assertj/core/util/diff/PostComparisonAction.java

@@ -1,10 +1,9 @@
 package com.trickl.assertj.core.util.diff;
 
 import java.util.List;
-
 import org.assertj.core.util.diff.Delta;
 
 @FunctionalInterface
 public interface PostComparisonAction {
   void apply(List<Delta<String>> diffs, String actual, String expected);
-}
+}

src/main/java/com/trickl/assertj/util/diff/JsonFieldDelta.java

@@ -33,10 +33,12 @@ public JsonFieldDelta(FieldComparisonFailure failure, TYPE type) {
   }
 
   @Override
-  public void applyTo(List<T> target) throws IllegalStateException {}
+  public void applyTo(List<T> target) throws IllegalStateException {
+  }
 
   @Override
-  public void verify(List<T> target) throws IllegalStateException {}
+  public void verify(List<T> target) throws IllegalStateException {
+  }
 
   @Override
   public String toString() {

src/main/java/com/trickl/assertj/util/diff/JsonRootDelta.java

@@ -25,11 +25,12 @@ public class JsonRootDelta<T> extends Delta<T> {
   @Getter private final TYPE type = TYPE.CHANGE;
 
   /**
-  * Create a delta for a JSON document.
-  * @param expected the expected element
-  * @param actual the actual element
-  * @param message the optional descriptive message
-  */
+   * Create a delta for a JSON document.
+   *
+   * @param expected the expected element
+   * @param actual the actual element
+   * @param message the optional descriptive message
+   */
   public JsonRootDelta(String expected, String actual, String message) {
     super(new Chunk(0, Collections.EMPTY_LIST), new Chunk(0, Collections.EMPTY_LIST));
     this.expected = expected;
@@ -38,10 +39,12 @@ public JsonRootDelta(String expected, String actual, String message) {
   }
 
   @Override
-  public void applyTo(List<T> target) throws IllegalStateException {}
+  public void applyTo(List<T> target) throws IllegalStateException {
+  }
 
   @Override
-  public void verify(List<T> target) throws IllegalStateException {}
+  public void verify(List<T> target) throws IllegalStateException {
+  }
 
   @Override
   public String toString() {