fix: bug with files ignore yaml patterns not filtering the list of files (#2304)

Co-authored-by: GitHub Action <[email protected]>

.github/workflows/test.yml

@@ -758,6 +758,24 @@ jobs:
         shell:
           bash
 
+      - name: Run changed-files with files_yaml, files_ignore_yaml
+        id: changed-files-ignore
+        uses: ./
+        with:
+          files_yaml: |
+            test:
+              - test/**.txt
+              - test/**.md
+          files_ignore_yaml: |
+            test:
+              - test/test.txt 
+
+      - name: Show output
+        run: |
+          echo "${{ toJSON(steps.changed-files-ignore.outputs) }}"
+        shell:
+          bash
+
       - name: Run changed-files with files_yaml, json and write_output_files
         id: changed-files-json-write-output-files
         uses: ./

dist/index.js

@@ -2743,6 +2743,10 @@ const getYamlFilePatterns = async ({ inputs, workingDirectory }) => {
                 core.warning(`files_ignore_yaml: Duplicated key ${key} detected, the ${filePatterns[key]} will be overwritten by ${newIgnoreFilePatterns[key]}.`);
             }
         }
+        filePatterns = {
+            ...filePatterns,
+            ...newIgnoreFilePatterns
+        };
     }
     if (inputs.filesIgnoreYamlFromSourceFile) {
         const inputFilesIgnoreYamlFromSourceFile = inputs.filesIgnoreYamlFromSourceFile

src/utils.ts

@@ -1284,6 +1284,10 @@ export const getYamlFilePatterns = async ({
         )
       }
     }
+    filePatterns = {
+      ...filePatterns,
+      ...newIgnoreFilePatterns
+    }
   }
 
   if (inputs.filesIgnoreYamlFromSourceFile) {

test/new.md

@@ -0,0 +1 @@
+This is a test markdown file

test/test.txt

@@ -1 +1 @@
-This is a test file.
+This is a test file...