Updated separator

bug: Fix command to narrow down target files (#330 )
* Fix INPUT_FILES_PATTERN * INPUT_FILES_PATTERN -> FILES_PATTERN
2022-02-16 22:54:26 -05:00 · 2022-01-26 02:32:28 -05:00 · 2022-01-25 23:18:02 -05:00 · 2022-01-25 23:15:26 -05:00 · 2022-01-25 23:10:53 -05:00 · 2022-01-25 22:41:38 -05:00
10 changed files with 227 additions and 73 deletions
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -60,6 +60,33 @@
      "contributions": [
        "doc"
      ]
+    },
+    {
+      "login": "Zamiell",
+      "name": "James",
+      "avatar_url": "https://avatars.githubusercontent.com/u/5511220?v=4",
+      "profile": "https://github.com/Zamiell",
+      "contributions": [
+        "doc"
+      ]
+    },
+    {
+      "login": "wushujames",
+      "name": "James Cheng",
+      "avatar_url": "https://avatars.githubusercontent.com/u/677529?v=4",
+      "profile": "https://github.com/wushujames",
+      "contributions": [
+        "doc"
+      ]
+    },
+    {
+      "login": "massongit",
+      "name": "Masaya Suzuki",
+      "avatar_url": "https://avatars.githubusercontent.com/u/15100604?v=4",
+      "profile": "https://qiita.com/SUZUKI_Masaya",
+      "contributions": [
+        "code"
+      ]
    }
  ],
  "contributorsPerLine": 7,
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -222,13 +222,22 @@ jobs:
          files: |
            .github/workflows/test.yml
            action.yml
+            **/test.txt
+            !test/test/test.txt
      - name: Show output
        run: |
          echo '${{ toJSON(steps.changed-files-specific.outputs) }}'
        shell:
          bash
+      - name: Check if a excluded file is not included in any_changed
+        if: "contains(steps.changed-files-specific.outputs.all_changed_files, 'test/test/test.txt')"
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific.outputs.all_changed_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_changed for specific files
-        if: "!contains(steps.changed-files-specific.outputs.all_changed_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.all_changed_files, '.github/workflows/test.yml')"
+        if: "!contains(steps.changed-files-specific.outputs.all_changed_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.all_changed_files, '.github/workflows/test.yml') && !contains(steps.changed-files-specific.outputs.all_changed_files, 'test/test.txt')"
        run: |
          if [[ "${{ steps.changed-files-specific.outputs.any_changed }}" != "false" ]]; then
            echo "Invalid output: Expected (false) got (${{ steps.changed-files-specific.outputs.any_changed }})"
@@ -236,8 +245,15 @@ jobs:
          fi
        shell:
          bash
+      - name: Check if a excluded file is not included in any_modified
+        if: "contains(steps.changed-files-specific.outputs.all_modified_files, 'test/test/test.txt')"
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific.outputs.all_modified_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_modified for specific files
-        if: "!contains(steps.changed-files-specific.outputs.all_modified_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.all_modified_files, '.github/workflows/test.yml')"
+        if: "!contains(steps.changed-files-specific.outputs.all_modified_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.all_modified_files, '.github/workflows/test.yml') && !contains(steps.changed-files-specific.outputs.all_modified_files, 'test/test.txt')"
        run: |
          if [[ "${{ steps.changed-files-specific.outputs.any_modified }}" != "false" ]]; then
            echo "Invalid output: Expected (false) got (${{ steps.changed-files-specific.outputs.any_modified }})"
@@ -245,8 +261,15 @@ jobs:
          fi
        shell:
          bash
+      - name: Check if a excluded file is not included in any_deleted
+        if: "contains(steps.changed-files-specific.outputs.deleted_files, 'test/test/test.txt')"
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific.outputs.deleted_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_deleted for specific files
-        if: "!contains(steps.changed-files-specific.outputs.deleted_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.deleted_files, '.github/workflows/test.yml')"
+        if: "!contains(steps.changed-files-specific.outputs.deleted_files, 'action.yml') && !contains(steps.changed-files-specific.outputs.deleted_files, '.github/workflows/test.yml') && !contains(steps.changed-files-specific.outputs.deleted_files, 'test/test.txt')"
        run: |
          if [[ "${{ steps.changed-files-specific.outputs.any_deleted }}" != "false" ]]; then
            echo "Invalid output: Expected (false) got (${{ steps.changed-files-specific.outputs.any_deleted }})"
@@ -426,12 +449,20 @@ jobs:
            test/changed-files-list.txt
          files: |
            **/workflows/rebase.yml
+      - name: Check if a excluded file is not included in any_changed
+        if: contains(steps.changed-files-specific-source-file.outputs.all_changed_files, 'test/test/test.txt')
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific-source-file.outputs.all_changed_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_changed from source files
        if: |
          (
            !contains(steps.changed-files-specific-source-file.outputs.all_changed_files, 'action.yml') &&
            !contains(steps.changed-files-specific-source-file.outputs.all_changed_files, '.github/workflows/test.yml') &&
-            !contains(steps.changed-files-specific-source-file.outputs.all_changed_files, '.github/workflows/rebase.yml')
+            !contains(steps.changed-files-specific-source-file.outputs.all_changed_files, '.github/workflows/rebase.yml') &&
+            !contains(steps.changed-files-specific-source-file.outputs.all_changed_files, 'test/test.txt')
          )
        run: |
          if [[ "${{ steps.changed-files-specific-source-file.outputs.any_changed }}" != "false" ]]; then
@@ -440,12 +471,20 @@ jobs:
          fi
        shell:
          bash
+      - name: Check if a excluded file is not included in any_modified
+        if: contains(steps.changed-files-specific-source-file.outputs.all_modified_files, 'test/test/test.txt')
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific-source-file.outputs.all_modified_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_modified from source files
        if: |
          (
            !contains(steps.changed-files-specific-source-file.outputs.all_modified_files, 'action.yml') &&
            !contains(steps.changed-files-specific-source-file.outputs.all_modified_files, '.github/workflows/test.yml') &&
-            !contains(steps.changed-files-specific-source-file.outputs.all_modified_files, '.github/workflows/rebase.yml')
+            !contains(steps.changed-files-specific-source-file.outputs.all_modified_files, '.github/workflows/rebase.yml') &&
+            !contains(steps.changed-files-specific-source-file.outputs.all_modified_files, 'test/test.txt')
          )
        run: |
          if [[ "${{ steps.changed-files-specific-source-file.outputs.any_modified }}" != "false" ]]; then
@@ -454,8 +493,15 @@ jobs:
          fi
        shell:
          bash
+      - name: Check if a excluded file is not included in any_deleted
+        if: contains(steps.changed-files-specific-source-file.outputs.deleted_files, 'test/test/test.txt')
+        run: |
+          echo "Invalid output: Expected not to include (test/test/test.txt) got (${{ steps.changed-files-specific-source-file.outputs.deleted_files }})"
+          exit 1
+        shell:
+          bash
      - name: Verify any_deleted from source files
-        if: "!contains(steps.changed-files-specific-source-file.outputs.deleted_files, 'action.yml') && !contains(steps.changed-files-specific-source-file.outputs.deleted_files, '.github/workflows/test.yml')"
+        if: "!contains(steps.changed-files-specific-source-file.outputs.deleted_files, 'action.yml') && !contains(steps.changed-files-specific-source-file.outputs.deleted_files, '.github/workflows/test.yml') && !contains(steps.changed-files-specific-source-file.outputs.deleted_files, 'test/test.txt')"
        run: |
          if [[ "${{ steps.changed-files-specific-source-file.outputs.any_deleted }}" != "false" ]]; then
            echo "Invalid output: Expected (false) got (${{ steps.changed-files-specific-source-file.outputs.any_deleted }})"
--- a/HISTORY.md
+++ b/HISTORY.md
@@ -1,5 +1,80 @@
 # Changelog

+## [v14](https://github.com/tj-actions/changed-files/tree/v14) (2022-01-26)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v4.0.0...v14)
+
+**Merged pull requests:**
+
+- Upgraded to v4.0.0 [\#328](https://github.com/tj-actions/changed-files/pull/328) ([jackton1](https://github.com/jackton1))
+
+## [v4.0.0](https://github.com/tj-actions/changed-files/tree/v4.0.0) (2022-01-26)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v13.2...v4.0.0)
+
+**Merged pull requests:**
+
+- Clean up variable name to reflect usage. [\#327](https://github.com/tj-actions/changed-files/pull/327) ([jackton1](https://github.com/jackton1))
+- Narrow down target files by exact match of INPUT\_FILES [\#326](https://github.com/tj-actions/changed-files/pull/326) ([massongit](https://github.com/massongit))
+- Upgraded to v13.2 [\#325](https://github.com/tj-actions/changed-files/pull/325) ([jackton1](https://github.com/jackton1))
+
+## [v13.2](https://github.com/tj-actions/changed-files/tree/v13.2) (2022-01-25)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v3.0.2...v13.2)
+
+**Merged pull requests:**
+
+- Upgraded to v3.0.2 [\#324](https://github.com/tj-actions/changed-files/pull/324) ([jackton1](https://github.com/jackton1))
+
+## [v3.0.2](https://github.com/tj-actions/changed-files/tree/v3.0.2) (2022-01-25)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v13.1...v3.0.2)
+
+**Fixed bugs:**
+
+- \[BUG\] wrong result of any\_change output [\#314](https://github.com/tj-actions/changed-files/issues/314)
+
+**Merged pull requests:**
+
+- Updated README.md [\#323](https://github.com/tj-actions/changed-files/pull/323) ([jackton1](https://github.com/jackton1))
+- docs: add massongit as a contributor for code [\#322](https://github.com/tj-actions/changed-files/pull/322) ([allcontributors[bot]](https://github.com/apps/allcontributors))
+- Deduplicate from files parameter without sorting [\#321](https://github.com/tj-actions/changed-files/pull/321) ([massongit](https://github.com/massongit))
+- docs: add wushujames as a contributor for doc [\#320](https://github.com/tj-actions/changed-files/pull/320) ([allcontributors[bot]](https://github.com/apps/allcontributors))
+- String literals need to be inside single-quotes [\#319](https://github.com/tj-actions/changed-files/pull/319) ([wushujames](https://github.com/wushujames))
+- Remove redundant debug line [\#318](https://github.com/tj-actions/changed-files/pull/318) ([jackton1](https://github.com/jackton1))
+- Updated README.md [\#317](https://github.com/tj-actions/changed-files/pull/317) ([jackton1](https://github.com/jackton1))
+- Upgraded to v13.1 [\#316](https://github.com/tj-actions/changed-files/pull/316) ([jackton1](https://github.com/jackton1))
+- Upgraded to v3.0.1 [\#315](https://github.com/tj-actions/changed-files/pull/315) ([jackton1](https://github.com/jackton1))
+
+## [v13.1](https://github.com/tj-actions/changed-files/tree/v13.1) (2022-01-09)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v3.0.1...v13.1)
+
+## [v3.0.1](https://github.com/tj-actions/changed-files/tree/v3.0.1) (2022-01-09)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v3.0.0...v3.0.1)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
+
+**Merged pull requests:**
+
+- Update tj-actions/glob action to v3.3 [\#313](https://github.com/tj-actions/changed-files/pull/313) ([renovate[bot]](https://github.com/apps/renovate))
+- Updated README.md [\#312](https://github.com/tj-actions/changed-files/pull/312) ([jackton1](https://github.com/jackton1))
+- Clean up unused code [\#311](https://github.com/tj-actions/changed-files/pull/311) ([jackton1](https://github.com/jackton1))
+- doc: add Zamiell as a contributor for doc [\#310](https://github.com/tj-actions/changed-files/pull/310) ([allcontributors[bot]](https://github.com/apps/allcontributors))
+- spelling/grammar [\#309](https://github.com/tj-actions/changed-files/pull/309) ([Zamiell](https://github.com/Zamiell))
+- Upgraded to v3.0.0 [\#308](https://github.com/tj-actions/changed-files/pull/308) ([jackton1](https://github.com/jackton1))
+
+## [v3.0.0](https://github.com/tj-actions/changed-files/tree/v3.0.0) (2022-01-05)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v13...v3.0.0)
+
+**Merged pull requests:**
+
+- Upgraded to v13 [\#307](https://github.com/tj-actions/changed-files/pull/307) ([jackton1](https://github.com/jackton1))
+
 ## [v13](https://github.com/tj-actions/changed-files/tree/v13) (2022-01-05)

 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v2.0.1...v13)
@@ -9,10 +84,6 @@
 - \[Feature\] Make it easier to exclude files [\#265](https://github.com/tj-actions/changed-files/issues/265)
 - \[Feature\] Support the same filter syntax as GitHub Workflows [\#264](https://github.com/tj-actions/changed-files/issues/264)

-**Closed issues:**
-
- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
-
 **Merged pull requests:**

 - Upgraded tj-actions/glob to v3.2 [\#306](https://github.com/tj-actions/changed-files/pull/306) ([jackton1](https://github.com/jackton1))
--- a/README.md
+++ b/README.md
@@ -7,7 +7,7 @@

 <!-- ALL-CONTRIBUTORS-BADGE:START - Do not remove or modify this section -->

-[![All Contributors](https://img.shields.io/badge/all_contributors-6-orange.svg?style=flat-square)](#contributors-)
+[![All Contributors](https://img.shields.io/badge/all_contributors-9-orange.svg?style=flat-square)](#contributors-)

 <!-- ALL-CONTRIBUTORS-BADGE:END -->

@@ -20,18 +20,18 @@ Retrieve all changed files relative to the default branch (`pull_request*` based
 *   Fast execution (0-2 seconds on average).
 *   Easy to debug.
 *   Boolean output indicating that certain files have been changed.
-*   Multiple repositories.
-*   Self hosted runners.
-*   GitHub Enterprise Server.
-*   Monorepos (Fetches only the last remote commit)
-*   Supports all platforms.
+*   Multiple repositories support.
+*   Runs on self hosted runners.
+*   GitHub Enterprise Server support.
+*   Monorepos (Fetches only the last remote commit).
+*   Supports all platforms (Linux, MacOS, Windows).
 *   List all files that have changed.
    *   Between the current pull request branch and the default branch.
    *   Between the last commit and the current pushed change.
    *   Between the last remote branch commit and the current HEAD.
 *   Restrict change detection to a subset of files.
    *   Report on files that have at least one change.
-    *   [Glob pattern](https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet) matching on a subset of files.
+    *   Using [Glob pattern](https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet) matching.

 ## Usage

@@ -39,7 +39,8 @@ Retrieve all changed files relative to the default branch (`pull_request*` based
 >
 > *   **IMPORTANT:** For `push` events you need to include `fetch-depth: 0` **OR** `fetch-depth: 2` depending on your use case.
 > *   When using `persist-credentials: false` with `actions/checkout@v2` you'll need to specify a `token` using the `token` input.
-> *   For monorepos were pulling all the branch history might not be desired the `fetch-depth` can be omitted for `pull_request` events.
+> *   For monorepos where pulling all the branch history might not be desired, you can omit `fetch-depth` for `pull_request` events.
+> *   For files located in a sub-directory ensure that the pattern specified contains `**` (globstar) to match any preceding directories or explicitly pass the full path relative to the project root. See: [#314](https://github.com/tj-actions/changed-files/issues/314)

 ```yaml
 name: CI
@@ -63,7 +64,7 @@ jobs:

      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14

      - name: List all changed files
        run: |
@@ -140,11 +141,11 @@ Support this project with a :star:

      - name: Get changed files using defaults
        id: changed-files
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14

      - name: Get changed files using a comma separator
        id: changed-files-comma
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          separator: ","

@@ -155,18 +156,18 @@ Support this project with a :star:
          done

      - name: Run step when a file changes
-        if: contains(steps.changed-files.outputs.modified_files, "my-file.txt")
+        if: contains(steps.changed-files.outputs.modified_files, 'my-file.txt')
        run: |
          echo "Your my-file.txt file has been modified."

      - name: Run step when a file has been deleted
-        if: contains(steps.changed-files.outputs.deleted_files, "test.txt")
+        if: contains(steps.changed-files.outputs.deleted_files, 'test.txt')
        run: |
          echo "Your test.txt file has been deleted."

      - name: Get specific changed files
        id: changed-files-specific
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          files: |
            my-file.txt
@@ -180,24 +181,24 @@ Support this project with a :star:
            **/migrate-*.sql

      - name: Run step if any of the listed files above change
-        if: steps.changed-files-specific.outputs.any_changed == "true"
+        if: steps.changed-files-specific.outputs.any_changed == 'true'
        run: |
          echo "One or more files listed above has changed."

      - name: Run step if only the files listed above change
-        if: steps.changed-files-specific.outputs.only_changed == "true"
+        if: steps.changed-files-specific.outputs.only_changed == 'true'
        run: |
          echo "Only files listed above have changed."

      - name: Run step if any of the listed files above is deleted
-        if: steps.changed-files.outputs.any_deleted == "true"
+        if: steps.changed-files.outputs.any_deleted == 'true'
        run: |
          for file in ${{ steps.changed-files.outputs.deleted_files }}; do
            echo "$file was deleted"
          done

      - name: Run step if all listed files above have been deleted
-        if: steps.changed-files.outputs.only_deleted == "true"
+        if: steps.changed-files.outputs.only_deleted == 'true'
        run: |
          for file in ${{ steps.changed-files.outputs.deleted_files }}; do
            echo "$file was deleted"
@@ -205,14 +206,14 @@ Support this project with a :star:

      - name: Use a source file or list of file(s) to populate to files input.
        id: changed-files-specific-source-file
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          files_from_source_file: |
            test/changed-files-list.txt

      - name: Use a source file or list of file(s) to populate to files input and optionally specify more files.
        id: changed-files-specific-source-file-and-specify-files
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          files_from_source_file: |
            test/changed-files-list.txt
@@ -221,13 +222,13 @@ Support this project with a :star:

      - name: Use a different commit SHA
        id: changed-files-custom-sha
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          sha: ${{ github.event.pull_request.head.sha }}

      - name: Use a different base SHA
        id: changed-files-custom-base-sha
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          base_sha: "2096ed0"
          
@@ -239,7 +240,7 @@ Support this project with a :star:

      - name: Run changed-files with defaults on the dir1
        id: changed-files-for-dir1
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          path: dir1

@@ -251,7 +252,7 @@ Support this project with a :star:

      - name: Run changed-files using the last commit on the remote branch
        id: changed-files-since-last-remote-commit
-        uses: tj-actions/changed-files@v13
+        uses: tj-actions/changed-files@v14
        with:
          since_last_remote_commit: "true"

@@ -273,6 +274,8 @@ Support this project with a :star:

 This package was created with [Cookiecutter](https://github.com/cookiecutter/cookiecutter).

+*   [tj-actions/glob](https://github.com/tj-actions/glob)
+
 ## Report Bugs

 Report bugs at https://github.com/tj-actions/changed-files/issues.
@@ -301,6 +304,11 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
    <td align="center"><a href="https://github.com/IvanPizhenko"><img src="https://avatars.githubusercontent.com/u/11859904?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Ivan Pizhenko</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=IvanPizhenko" title="Code">💻</a> <a href="https://github.com/tj-actions/changed-files/commits?author=IvanPizhenko" title="Documentation">📖</a></td>
    <td align="center"><a href="https://github.com/talva-tr"><img src="https://avatars.githubusercontent.com/u/82046981?v=4?s=100" width="100px;" alt=""/><br /><sub><b>talva-tr</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=talva-tr" title="Code">💻</a></td>
    <td align="center"><a href="https://bandism.net/"><img src="https://avatars.githubusercontent.com/u/22633385?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Ikko Ashimine</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=eltociear" title="Documentation">📖</a></td>
+    <td align="center"><a href="https://github.com/Zamiell"><img src="https://avatars.githubusercontent.com/u/5511220?v=4?s=100" width="100px;" alt=""/><br /><sub><b>James</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=Zamiell" title="Documentation">📖</a></td>
+  </tr>
+  <tr>
+    <td align="center"><a href="https://github.com/wushujames"><img src="https://avatars.githubusercontent.com/u/677529?v=4?s=100" width="100px;" alt=""/><br /><sub><b>James Cheng</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=wushujames" title="Documentation">📖</a></td>
+    <td align="center"><a href="https://qiita.com/SUZUKI_Masaya"><img src="https://avatars.githubusercontent.com/u/15100604?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Masaya Suzuki</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=massongit" title="Code">💻</a></td>
  </tr>
 </table>

--- a/action.yml
+++ b/action.yml
@@ -99,21 +99,6 @@ outputs:
 runs:
  using: 'composite'
  steps:
-    - run: |
-        bash $GITHUB_ACTION_PATH/sourcefiles.sh
-      id: source-input-files
-      shell: bash
-      env:
-        # INPUT_<VARIABLE_NAME> is not available in Composite run steps
-        # https://github.community/t/input-variable-name-is-not-available-in-composite-run-steps/127611
-        INPUT_FILES: ${{ inputs.files }}
-        INPUT_FILES_FROM_SOURCE_FILE: ${{ inputs.files_from_source_file }}
-    - name: Glob match
-      uses: tj-actions/glob@v3.2
-      id: glob
-      with:
-        files: ${{ steps.source-input-files.outputs.files }}
-        files-separator: " "
    - run: |
        # "Set base sha..."
        if [[ -n "${{ inputs.base_sha }}" ]]; then
@@ -127,6 +112,22 @@ runs:
        fi
      id: base-sha
      shell: bash
+    - run: |
+        bash $GITHUB_ACTION_PATH/sourcefiles.sh
+      id: source-input-files
+      shell: bash
+      env:
+        # INPUT_<VARIABLE_NAME> is not available in Composite run steps
+        # https://github.community/t/input-variable-name-is-not-available-in-composite-run-steps/127611
+        INPUT_FILES: ${{ inputs.files }}
+        INPUT_FILES_FROM_SOURCE_FILE: ${{ inputs.files_from_source_file }}
+    - name: Glob match
+      uses: tj-actions/glob@v3.3
+      id: glob
+      with:
+        files: ${{ steps.source-input-files.outputs.files }}
+        files-separator: " "
+        separator: "|"
    - run: |
        bash $GITHUB_ACTION_PATH/entrypoint.sh
      id: changed-files
@@ -140,7 +141,7 @@ runs:
        INPUT_SHA: ${{ inputs.sha }}
        INPUT_BASE_SHA: ${{ steps.base-sha.outputs.base_sha }}
        INPUT_TOKEN: ${{ inputs.token }}
-        INPUT_FILES: ${{ steps.glob.outputs.paths }}
+        INPUT_FILES_PATTERN: ${{ steps.glob.outputs.paths }}
        INPUT_SEPARATOR: ${{ inputs.separator }}
        INPUT_PATH: ${{ inputs.path }}

--- a/entrypoint.sh
+++ b/entrypoint.sh
@@ -93,7 +93,7 @@ echo "Retrieving changes between $PREVIOUS_SHA ($TARGET_BRANCH) → $CURRENT_SHA

 echo "Getting diff..."

-if [[ -z "${INPUT_FILES[*]}" ]]; then
+if [[ -z "$INPUT_FILES_PATTERN" ]]; then
  ADDED=$(git diff --diff-filter=A --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
  COPIED=$(git diff --diff-filter=C --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
  DELETED=$(git diff --diff-filter=D --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
@@ -106,25 +106,23 @@ if [[ -z "${INPUT_FILES[*]}" ]]; then
  ALL_CHANGED=$(git diff --diff-filter="ACMR" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
  ALL_MODIFIED=$(git diff --diff-filter="ACMRD" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
 else
-  echo "Input files: ${INPUT_FILES[*]}"
+  echo "Input files pattern: $INPUT_FILES_PATTERN"
+  FILES_PATTERN="^($INPUT_FILES_PATTERN)$"

-  FILES=$(echo "${INPUT_FILES[*]}" | awk '{gsub(/ /,"\n"); print $0;}' | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-
-  echo "Checking for file changes: \"${FILES}\"..."
-  ADDED=$(git diff --diff-filter=A --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  COPIED=$(git diff --diff-filter=C --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  DELETED=$(git diff --diff-filter=D --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  MODIFIED=$(git diff --diff-filter=M --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  RENAMED=$(git diff --diff-filter=R --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  TYPE_CHANGED=$(git diff --diff-filter=T --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  UNMERGED=$(git diff --diff-filter=U --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  UNKNOWN=$(git diff --diff-filter=X --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  ALL_CHANGED_AND_MODIFIED=$(git diff --diff-filter="*ACDMRTUX" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  ALL_CHANGED=$(git diff --diff-filter="ACMR" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  ALL_MODIFIED=$(git diff --diff-filter="ACMRD" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "(${FILES})" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  ADDED=$(git diff --diff-filter=A --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  COPIED=$(git diff --diff-filter=C --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  DELETED=$(git diff --diff-filter=D --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  MODIFIED=$(git diff --diff-filter=M --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  RENAMED=$(git diff --diff-filter=R --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  TYPE_CHANGED=$(git diff --diff-filter=T --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  UNMERGED=$(git diff --diff-filter=U --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  UNKNOWN=$(git diff --diff-filter=X --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  ALL_CHANGED_AND_MODIFIED=$(git diff --diff-filter="*ACDMRTUX" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  ALL_CHANGED=$(git diff --diff-filter="ACMR" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  ALL_MODIFIED=$(git diff --diff-filter="ACMRD" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | grep -E "$FILES_PATTERN" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')

  ALL_OTHER_CHANGED=$(git diff --diff-filter="ACMR" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  UNIQUE_ALL_CHANGED=$(echo "${ALL_CHANGED}" | awk '{gsub(/\|/,"\n"); print $0;}' | sort -u | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  UNIQUE_ALL_CHANGED=$(echo "${ALL_CHANGED}" | awk '{gsub(/\|/,"\n"); print $0;}' | awk '!a[$0]++' | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')

  if [[ -n "${UNIQUE_ALL_CHANGED}" ]]; then
    echo "Matching changed files: ${UNIQUE_ALL_CHANGED}"
@@ -154,7 +152,7 @@ else
  fi

  ALL_OTHER_MODIFIED=$(git diff --diff-filter="ACMRD" --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  UNIQUE_ALL_MODIFIED=$(echo "${ALL_MODIFIED}" | awk '{gsub(/\|/,"\n"); print $0;}' | sort -u | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  UNIQUE_ALL_MODIFIED=$(echo "${ALL_MODIFIED}" | awk '{gsub(/\|/,"\n"); print $0;}' | awk '!a[$0]++' | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')

  if [[ -n "${UNIQUE_ALL_MODIFIED}" ]]; then
    echo "Matching modified files: ${UNIQUE_ALL_MODIFIED}"
@@ -184,7 +182,7 @@ else
  fi

  ALL_OTHER_DELETED=$(git diff --diff-filter=D --name-only "$PREVIOUS_SHA" "$CURRENT_SHA" | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
-  UNIQUE_ALL_DELETED=$(echo "${DELETED}" | awk '{gsub(/\|/,"\n"); print $0;}' | sort -u | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+  UNIQUE_ALL_DELETED=$(echo "${DELETED}" | awk '{gsub(/\|/,"\n"); print $0;}' | awk '!a[$0]++' | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')

  if [[ -n "${UNIQUE_ALL_DELETED}" ]]; then
    echo "Matching deleted files: ${UNIQUE_ALL_DELETED}"
--- a/sourcefiles.sh
+++ b/sourcefiles.sh
@@ -15,13 +15,13 @@ if [[ -n $INPUT_FILES_FROM_SOURCE_FILE ]]; then
  done
 fi

-IFS=" " read -r -a CLEAN_FILES <<< "$(echo "${RAW_FILES[*]}" | tr "\r\n" "\n" | tr " " "\n" | sort -u | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"
+IFS=" " read -r -a CLEAN_FILES <<< "$(echo "${RAW_FILES[*]}" | tr "\r\n" "\n" | tr " " "\n" | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"

-IFS=" " read -r -a CLEAN_INPUT_FILES <<< "$(echo "${INPUT_FILES}" | tr "\r\n" "\n" | tr " " "\n" | sort -u | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"
+IFS=" " read -r -a CLEAN_INPUT_FILES <<< "$(echo "${INPUT_FILES}" | tr "\r\n" "\n" | tr " " "\n" | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"

 FILES=("${CLEAN_FILES[@]}" "${CLEAN_INPUT_FILES[@]}")

-IFS=" " read -r -a ALL_UNIQUE_FILES <<< "$(echo "${FILES[@]}" | tr "\r\n" "\n" | tr " " "\n" | sort -u | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"
+IFS=" " read -r -a ALL_UNIQUE_FILES <<< "$(echo "${FILES[@]}" | tr "\r\n" "\n" | tr " " "\n" | awk '!a[$0]++' | awk -v d=" " '{s=(NR==1?s:s d)$0}END{print s}')"

 echo "Input files: ${ALL_UNIQUE_FILES[*]}"

--- a/test/changed-files-list.txt
+++ b/test/changed-files-list.txt
@@ -2,4 +2,5 @@
 action.yml
 action.yml
 action.yml
-!*.txt
+**/test.txt
+!test/test/test.txt
--- a/test/test.txt
+++ b/test/test.txt
@@ -0,0 +1 @@
+This is a test file
--- a/test/test/test.txt
+++ b/test/test/test.txt
@@ -0,0 +1 @@
+This is a test file
Author	SHA1	Message	Date
Tonye Jack	fb063fc7d4	Updated separator	2022-02-16 22:54:26 -05:00
Masaya Suzuki	e9f69ac719	bug: Fix command to narrow down target files (#330 ) * Fix INPUT_FILES_PATTERN * INPUT_FILES_PATTERN -> FILES_PATTERN	2022-01-26 02:32:28 -05:00
Tonye Jack	416c6348f8	Upgraded to v14 (#329 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-25 23:18:02 -05:00
Tonye Jack	75f52eeda4	Upgraded to v4.0.0 (#328 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-25 23:15:26 -05:00
Tonye Jack	20f6f68d34	Clean up variable name to reflect usage. (#327 )	2022-01-25 23:10:53 -05:00
Masaya Suzuki	b94745bfeb	Narrow down target files by exact match of INPUT_FILES (#326 )	2022-01-25 22:41:38 -05:00
Tonye Jack	00cfbecfee	Upgraded from v3.0.2 -> v13.2 (#325 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-24 21:12:14 -05:00
Tonye Jack	092bd2feb9	Upgraded to v3.0.2 (#324 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-24 21:02:14 -05:00
Tonye Jack	738523bd5f	Updated README.md (#323 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-24 20:52:31 -05:00
allcontributors[bot]	910453353b	docs: add massongit as a contributor for code (#322 ) Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>	2022-01-24 20:51:13 -05:00
Masaya Suzuki	13328439bc	Deduplicate from files parameter without sorting (#321 ) Co-authored-by: Tonye Jack <jtonye@ymail.com>	2022-01-24 20:50:47 -05:00
allcontributors[bot]	7fe764fa81	docs: add wushujames as a contributor for doc (#320 ) * docs: update README.md [skip ci] * docs: update .all-contributorsrc [skip ci] Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>	2022-01-21 11:20:50 -05:00
James Cheng	204ed812d0	String literals need to be inside single-quotes (#319 ) * String literals need to be inside single-quotes Per https://docs.github.com/en/actions/learn-github-actions/expressions#literals string You must use single quotes. Escape literal single-quotes with a single quote. * Update README.md Co-authored-by: Tonye Jack <jtonye@ymail.com>	2022-01-21 11:19:23 -05:00
Tonye Jack	7dfde9f9fe	Remove redundant debug line (#318 )	2022-01-20 01:53:39 -05:00
Tonye Jack	269e6c2a15	Update README.md	2022-01-19 18:51:22 -05:00
Tonye Jack	80cb5a3de4	Update README.md	2022-01-15 02:08:17 -05:00
Tonye Jack	61ee456a9d	Updated README.md (#317 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-10 15:05:01 -05:00
Tonye Jack	b6dd719334	Update README.md	2022-01-10 14:42:33 -05:00
Tonye Jack	40292bd765	Update README.md	2022-01-09 14:30:17 -05:00
Tonye Jack	64580b3060	Upgraded to v13.1 (#316 ) * Upgraded from v3.0.0 -> v13.1 * Update README.md Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-09 14:02:00 -05:00
Tonye Jack	e5db4109e0	Upgraded from v3.0.0 -> v3.0.1 (#315 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-09 13:54:05 -05:00
Tonye Jack	98bf782e42	Update README.md	2022-01-09 12:11:39 -05:00
Tonye Jack	e77fe6304b	Update README.md	2022-01-09 12:10:47 -05:00
Renovate Bot	d0d33c82a2	Update tj-actions/glob action to v3.3	2022-01-09 04:49:33 -05:00
Tonye Jack	4ea1cf7500	Updated README.md (#312 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-09 03:23:45 -05:00
Tonye Jack	835db067d1	Clean up unused code (#311 ) * Clean up unused code * Update entrypoint.sh * Update action.yml * Update entrypoint.sh * Update action.yml	2022-01-09 03:18:29 -05:00
allcontributors[bot]	281391b987	doc: add Zamiell as a contributor for doc (#310 ) * docs: update README.md [skip ci] * docs: update .all-contributorsrc [skip ci] * Update .all-contributorsrc * Update README.md Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Co-authored-by: Tonye Jack <jtonye@ymail.com>	2022-01-07 14:31:32 -05:00
James	e458dee210	spelling/grammar (#309 )	2022-01-07 13:51:45 -05:00
Tonye Jack	c4d765fc77	Update README.md	2022-01-05 00:19:25 -05:00
Tonye Jack	283e1f384e	Upgraded from v13 -> v3.0.0 (#308 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-01-04 22:56:23 -05:00