Add checkstyle rules that we are already following

application/src/test/java/org/opentripplanner/framework/csv/parser/AbstractCsvParserTest.java

@@ -157,7 +157,9 @@ void testLocalDateParsingIssue() {
     );
 
     subject.headersMatch();
-    while (subject.hasNext()) {}
+    while (subject.hasNext()) {
+      // Spinn trough all items so we can test the result
+    }
 
     var error1 = issueStore.listIssues().getFirst();
     var error2 = issueStore.listIssues().getLast();

checkstyle.xml

@@ -5,9 +5,14 @@
   "https://checkstyle.org/dtds/configuration_1_3.dtd">
 <module name="Checker">
     <module name="TreeWalker">
+        <module name="ArrayTypeStyle"/>
         <module name="AvoidStarImport"/>
+        <module name="DefaultComesLast"/>
+        <module name="EmptyStatement"/>
         <module name="NeedBraces"/>
         <module name="RedundantImport"/>
+        <module name="SimplifyBooleanReturn"/>
+        <module name="StringLiteralEquality"/>
         <module name="TrailingComment">
             <property name="format" value="^\s*$"/>
         </module>

openrewrite.yml

@@ -3,6 +3,9 @@ name: org.opentripplanner.CodeStyle
 recipeList:
   # Note, these recipes should be listed in the order they should be applied.
   # However, even a single recipe can cause a need to run rewrite twice.
+  - org.openrewrite.staticanalysis.DefaultComesLast
   - org.openrewrite.staticanalysis.NeedBraces
+  - org.openrewrite.staticanalysis.EmptyBlock
+  - org.openrewrite.staticanalysis.StringLiteralEquality
   - org.openrewrite.staticanalysis.RemoveUnusedLocalVariables
   - org.openrewrite.java.RemoveUnusedImports