#184 Checking another way of getting PR changelog - via an external github action

Author: Roman Glushko

.github/workflows/gradle.yml

@@ -24,13 +24,14 @@ jobs:
         run: chmod +x gradlew
 #      - name: Run automated tests
 #        run: ./gradlew test -i
-      - name: Pull Base Branch
-        run: git fetch origin ${{ github.base_ref }}:${{ github.base_ref }}
+      - uses: trilom/file-changes-action@v1.2.3
+      - name: Run Code Style Check
+        run: |
+          echo 'Modified Files: ${{ steps.file_changes.outputs.files}}'
       - name: Run Code Style Check
         run: ./gradlew checkstyleCI -i
         env:
-          TARGET_BRANCH: ${{ github.base_ref }}
-          WORKING_BRANCH: ${{ github.head_ref }}
+          MODIFIED_FILES: ${{ steps.file_changes.outputs.files}}
       - name: Run PMD Quality Check
         run: ./gradlew pmdMain -i
       - name: Run Spotbugs Quality Check
@@ -80,6 +81,8 @@ jobs:
         run: chmod +x gradlew
       - name: Pull Base Branch
         run: git fetch origin ${{ github.base_ref }}:${{ github.base_ref }}
+      - name: Pull Working Branch
+        run: git fetch origin ${{ github.base_ref }}:${{ github.base_ref }}
 #      - name: Run automated tests
 #        run: ./gradlew test -i
       - name: Run Code Style Check

gradle-tasks/staticChecks.gradle

@@ -65,43 +65,46 @@ task checkstyleCI(type: Checkstyle) {
  */
 def getChangedFiles() {
     // Get the target and source branch
-    def envTargetBranch = System.getenv("TARGET_BRANCH")
-    def envSourceBranch = System.getenv("WORKING_BRANCH")
+    //def envTargetBranch = System.getenv("TARGET_BRANCH")
+    //def envSourceBranch = System.getenv("WORKING_BRANCH")
+    def modifiedFiles = System.getenv("MODIFIED_FILES")
 
-    println "Target Branch: ${envTargetBranch}"
-    println "Source Branch: ${envSourceBranch}"
+    println "Modified Files: ${modifiedFiles}"
 
-    // Compare to master if no branch specified
-    def targetBranch
-    if (project.hasProperty("branch")) {
-        targetBranch = project.property("branch")
-    } else {
-        targetBranch = envTargetBranch ? "origin/${envTargetBranch}" : getParentBranch()
-    }
-    def sourceBranch = envSourceBranch ? "origin/${envSourceBranch}" : ""
+    //println "Target Branch: ${envTargetBranch}"
+    //println "Source Branch: ${envSourceBranch}"
 
-    println "Merging ${sourceBranch} into ${targetBranch}"
+    // Compare to master if no branch specified
+//    def targetBranch
+//    if (project.hasProperty("branch")) {
+//        targetBranch = project.property("branch")
+//    } else {
+//        targetBranch = envTargetBranch ? "origin/${envTargetBranch}" : getParentBranch()
+//    }
+//    def sourceBranch = envSourceBranch ? "origin/${envSourceBranch}" : ""
+//
+//    println "Merging ${sourceBranch} into ${targetBranch}"
 
     // Get list of all changed files including status
-    def systemOutStream = new ByteArrayOutputStream()
-    def command = "git diff --name-status --diff-filter=dr $targetBranch $sourceBranch"
-    command.execute().waitForProcessOutput(systemOutStream, System.err)
-    def allFiles = systemOutStream.toString().trim().split('\n')
-    systemOutStream.close()
-
-    // Remove the status prefix
-    Pattern statusPattern = Pattern.compile("(\\w)\\t+(.+)")
+//    def systemOutStream = new ByteArrayOutputStream()
+//    def command = "git diff --name-status --diff-filter=dr $targetBranch $sourceBranch"
+//    command.execute().waitForProcessOutput(systemOutStream, System.err)
+//    def allFiles = systemOutStream.toString().trim().split('\n')
+//    systemOutStream.close()
+//
+//    // Remove the status prefix
+//    Pattern statusPattern = Pattern.compile("(\\w)\\t+(.+)")
     List<String> files = new ArrayList<>()
 
-    println "Changed files:"
+    //println "Changed files:"
 
-    for (file in allFiles) {
-        Matcher matcher = statusPattern.matcher(file)
-        if (matcher.find()) {
-            println "+ ${matcher.group(2)}"
-            files.add(matcher.group(2))
-        }
-    }
+//    for (file in allFiles) {
+//        Matcher matcher = statusPattern.matcher(file)
+//        if (matcher.find()) {
+//            println "+ ${matcher.group(2)}"
+//            files.add(matcher.group(2))
+//        }
+//    }
 
     // Return the list of touched files
     files