Move slf4j ban to checkstyle

Author: me4502

config/checkstyle/checkstyle-suppression.xml

@@ -9,8 +9,6 @@
     <suppress files=".*[\\/]test[\\/].*\.java" checks="package-info-NullMarked|JavadocPackage"/>
     <!-- Suppress ICs for now -->
     <suppress files=".*[\\/]mechanics[\\/]ic[\\/].*\.java" checks=".*"/>
-    <!-- Suppress doctools -->
-    <suppress files=".*[\\/]doctools[\\/].*\.java" checks=".*"/>
     <!-- Suppress legacy files -->
     <suppress files=".*[\\/]ItemUtil.java" checks=".*"/>
     <suppress files=".*[\\/]ICUtil.java" checks=".*"/>

config/checkstyle/checkstyle.xml

@@ -145,6 +145,10 @@ Checks based on Google Checks, modified for EngineHub.
             <property name="sortImportsInGroupAlphabetically" value="true"/>
             <property name="customImportOrderRules" value="THIRD_PARTY_PACKAGE###STANDARD_JAVA_PACKAGE###STATIC"/>
         </module>
+        <module name="IllegalImport">
+            <property name="illegalPkgs" value="org.slf4j"/>
+            <message key="name.invalidPattern" value="Illegal import - {0}. Use Log4J instead."/>
+        </module>
         <module name="MethodParamPad"/>
         <module name="OperatorWrap">
             <property name="option" value="NL"/>

craftbook-bukkit/doctools/src/main/java/org/enginehub/craftbook/internal/util/CommandUtils.java

@@ -140,8 +140,7 @@ private static String cmdsToPerms(List<Command> cmds, String prefix) {
                         .stream()
                         .sorted(Comparator.comparing(Command::getName))
                         .toList(),
-                    prefix + c.getName() + " ")
-                );
+                    prefix + c.getName() + " "));
         });
         return permsOutput.toString();
     }