Removed checkstyle and PMD configurations

Author: VitaliyBoyko

.github/workflows/gradle.yml

@@ -90,71 +90,18 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
       - name: Set up JDK 17
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           java-version: 17
           distribution: 'temurin'
           cache: gradle
-      - uses: actions/cache@v3
+      - name: Qodana - Code Quality
+        uses: JetBrains/[email protected]
         with:
-          path: |
-            ~/.gradle/caches
-            ~/.gradle/wrapper
-          key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*') }}
-          restore-keys: |
-            ${{ runner.os }}-gradle-
-      - name: Gradle wrapper
-        run: gradle wrapper
-      - name: Grant execute permission for gradlew
-        run: chmod +x gradlew
-      - id: file_changes
-        uses: trilom/[email protected]
-      - name: Filter Java Files
-        id: filter_java_files
-        run: |
-          # Get modified and added files
-          MODIFIED_FILES=${{ steps.file_changes.outputs.files_modified }}
-          ADDED_FILES=${{ steps.file_changes.outputs.files_added }}
-
-          echo "Modified files: $MODIFIED_FILES"
-          echo "Added files: $ADDED_FILES"
-
-          # Simple approach using shell commands to filter Java files
-          # Remove brackets and quotes from the JSON arrays
-          MODIFIED_FILES=$(echo "$MODIFIED_FILES" | sed 's/^\[//;s/\]$//;s/"//g;s/,/ /g')
-          ADDED_FILES=$(echo "$ADDED_FILES" | sed 's/^\[//;s/\]$//;s/"//g;s/,/ /g')
-
-          # Combine the files and filter for .java files
-          ALL_FILES="$MODIFIED_FILES $ADDED_FILES"
-          JAVA_FILES=""
-
-          # If there are files to process
-          if [ ! -z "$ALL_FILES" ]; then
-            # Filter for Java files and format as JSON array
-            for file in $ALL_FILES; do
-              if [[ "$file" == *.java ]]; then
-                if [ -z "$JAVA_FILES" ]; then
-                  JAVA_FILES="\"$file\""
-                else
-                  JAVA_FILES="$JAVA_FILES,\"$file\""
-                fi
-              fi
-            done
-          fi
-
-          # Create a proper JSON array
-          JAVA_FILES="[$JAVA_FILES]"
-
-          echo "JAVA_FILES=$JAVA_FILES" >> $GITHUB_ENV
-          echo "Filtered Java Files: $JAVA_FILES"
-      - name: Run Code Style Check
-        run: ./gradlew checkstyleCI --no-daemon
-        env:
-          MODIFIED_FILES: ${{ env.JAVA_FILES }}
-          ACTIONS_STEP_DEBUG: true
-      - name: Run PMD Quality Check
-        run: ./gradlew pmdCI --no-daemon
-        env:
-          MODIFIED_FILES: ${{ env.JAVA_FILES }}
+          cache-default-branch-only: true
+          use-caches: 'gradle'
+          args: '--baseline,qodana.sarif.json'

build.gradle.kts

@@ -3,15 +3,12 @@
  * See COPYING.txt for license details.
  */
 
-import groovy.json.JsonSlurper
 import org.jetbrains.changelog.Changelog
 import org.jetbrains.changelog.markdownToHTML
 import org.jetbrains.intellij.platform.gradle.TestFrameworkType
 
 plugins {
     id("java")
-    id("checkstyle")
-    id("pmd")
     alias(libs.plugins.kotlin)
     alias(libs.plugins.intelliJPlatform)
     alias(libs.plugins.changelog)
@@ -150,19 +147,7 @@ tasks {
         useJUnitPlatform()
     }
 
-    checkstyle {
-        toolVersion = "8.31"
-        isIgnoreFailures = false
-        maxWarnings = 0
-        configFile = rootProject.file("${rootDir}/gradle-tasks/checkstyle/checkstyle.xml")
-    }
 
-    pmd {
-        toolVersion = "6.21.0"
-        isConsoleOutput = true
-        ruleSetFiles = files("${rootDir}/gradle-tasks/pmd/ruleset.xml")
-        ruleSets = listOf()
-    }
 }
 
 intellijPlatformTesting {
@@ -188,68 +173,12 @@ intellijPlatformTesting {
     }
 }
 
-// Configure Checkstyle tasks
-tasks.withType(Checkstyle::class).configureEach {
-    // Specify all files that should be checked
-    classpath = files()
-    setSource("${project.rootDir}")
-}
-
-// Execute Checkstyle on all files
-tasks.register<Checkstyle>("checkstyle") {
-    // Task-specific configuration can go here if necessary
-}
-
-// Execute Checkstyle on all modified files
-tasks.register<Checkstyle>("checkstyleCI") {
-    val changedFiles = getChangedFiles()
-    include(changedFiles)
-}
-
-// Configure PMD tasks
-tasks.withType(Pmd::class).configureEach {
-    // Specify all files that should be checked
-    classpath = files()
-    setSource("${project.rootDir}")
-}
-
-// Execute PMD on all files
-tasks.register<Pmd>("pmd") {
-    // Task-specific configuration can go here if necessary
-}
 
-// Execute PMD on all modified files
-tasks.register<Pmd>("pmdCI") {
-    val changedFiles = getChangedFiles()
-    include(changedFiles)
-}
 
-/**
- * Get all files that are changed but not deleted nor renamed.
- * Compares to master or the specified target branch.
- *
- * @return list of all changed files
- */
-fun getChangedFiles(): List<String> {
-    val modifiedFilesJson = System.getenv("MODIFIED_FILES")
-    val files = mutableListOf<String>()
 
-    if (modifiedFilesJson == null) {
-        return files
-    }
-
-    println("Modified Files: $modifiedFilesJson")
 
-    // Parse the JSON string into a list of files
-    val modifiedFiles = JsonSlurper().parseText(modifiedFilesJson) as List<*>
 
-    modifiedFiles.forEach {
-        files.add(it.toString())
-    }
 
-    // Return the list of touched files
-    return files
-}
 
 kover {
     currentProject {