.github/workflows/bld.yml

@@ -12,12 +12,12 @@ jobs:
 
     steps:
       - name: Checkout source repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0
 
       - name: Set up JDK ${{ matrix.java-version }}
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v3
         with:
           distribution: "zulu"
           java-version: ${{ matrix.java-version }}

.github/workflows/pages.yml

@@ -30,14 +30,14 @@ jobs:
 
     steps:
       - name: Checkout source repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0
 
       - name: Set up JDK 17
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v3
         with:
-          distribution: "zulu"
+          distribution: 'zulu'
           java-version: 17
 
       - name: Build Javadocs
@@ -50,7 +50,7 @@ jobs:
         uses: actions/upload-pages-artifact@v1
         with:
           # Upload generated Javadocs repository
-          path: "build/javadoc/"
+          path: 'build/javadoc/'
 
       - name: Deploy to GitHub Pages
         id: deployment

config/pmd.xml

@@ -19,15 +19,15 @@
         </properties>
     </rule>
 
-
     <!-- CODE STYLE -->
     <rule ref="category/java/codestyle.xml">
         <exclude name="AtLeastOneConstructor"/>
         <exclude name="ClassNamingConventions"/>
-        <exclude name="CommentDefaultAccessModifier"/>
         <exclude name="ConfusingTernary"/>
+        <exclude name="CommentDefaultAccessModifier"/>
         <exclude name="FieldNamingConventions"/>
         <exclude name="LocalVariableCouldBeFinal"/>
+        <exclude name="LocalVariableNamingConventions"/>
         <exclude name="LongVariable"/>
         <exclude name="MethodArgumentCouldBeFinal"/>
         <exclude name="OnlyOneReturn"/>
@@ -35,9 +35,8 @@
         <exclude name="ShortClassName"/>
         <exclude name="ShortMethodName"/>
         <exclude name="ShortVariable"/>
-        <exclude name="UseExplicitTypes"/>
-        <exclude name="UseUnderscoresInNumericLiterals"/>
         <exclude name="UselessParentheses"/>
+        <exclude name="UseUnderscoresInNumericLiterals"/>
     </rule>
 
     <rule ref="category/java/codestyle.xml/UnnecessaryImport">
@@ -53,6 +52,8 @@
         <exclude name="AvoidUncheckedExceptionsInSignatures"/>
         <exclude name="CognitiveComplexity"/>
         <exclude name="CyclomaticComplexity"/>
+        <exclude name="ExcessiveClassLength"/>
+        <exclude name="ExcessiveMethodLength"/>
         <exclude name="ExcessiveParameterList"/>
         <exclude name="ExcessivePublicCount"/>
         <exclude name="GodClass"/>

src/bld/java/rife/bld/extension/DetektOperationBuild.java

@@ -34,7 +34,7 @@ public class DetektOperationBuild extends Project {
     public DetektOperationBuild() {
         pkg = "rife.bld.extension";
         name = "DetektOperation";
-        version = version(0, 9, 4, "SNAPSHOT");
+        version = version(0, 9, 3);
 
         javaRelease = 17;
         downloadSources = true;
@@ -72,7 +72,7 @@ public class DetektOperationBuild extends Project {
                 .license(
                         new PublishLicense()
                                 .name("The Apache License, Version 2.0")
-                                .url("https://www.apache.org/licenses/LICENSE-2.0.txt")
+                                .url("http://www.apache.org/licenses/LICENSE-2.0.txt")
                 )
                 .scm(
                         new PublishScm()

src/main/java/rife/bld/extension/DetektOperation.java

@@ -18,11 +18,8 @@ package rife.bld.extension;
 
 import rife.bld.BaseProject;
 import rife.bld.operations.AbstractProcessOperation;
-import rife.bld.operations.exceptions.ExitStatusException;
-import rife.tools.exceptions.FileUtilsErrorException;
 
 import java.io.File;
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
@@ -244,22 +241,6 @@ public class DetektOperation extends AbstractProcessOperation<DetektOperation> {
         return this;
     }
 
-    /**
-     * Performs the operation.
-     *
-     * @throws InterruptedException    when the operation was interrupted
-     * @throws IOException             when an exception occurred during the execution of the process
-     * @throws FileUtilsErrorException when an exception occurred during the retrieval of the operation output
-     * @throws ExitStatusException     when the exit status was changed during the operation
-     */
-    @Override
-    public void execute() throws IOException, FileUtilsErrorException, InterruptedException, ExitStatusException {
-        super.execute();
-        if (successful_ && LOGGER.isLoggable(Level.INFO)) {
-            LOGGER.info("Detekt executed successfully.");
-        }
-    }
-
     /**
      * Part of the {@link #execute} operation, constructs the command list
      * to use for building the process.