Fixed unbound variable warnings (#296)

Co-authored-by: jackton1 <jackton1@users.noreply.github.com>

.github/workflows/test.yml

@@ -18,7 +18,7 @@ jobs:
       - name: Checkout to branch
         uses: actions/checkout@v2
       - name: shellcheck
-        uses: reviewdog/action-shellcheck@v1.12
+        uses: reviewdog/action-shellcheck@v1.13
 
   test-multiple-repositories:
     name: Test with multiple repositories

HISTORY.md

@@ -1,16 +1,44 @@
 # Changelog
 
-## [v12](https://github.com/tj-actions/changed-files/tree/v12) (2021-12-14)
+## [v12.1](https://github.com/tj-actions/changed-files/tree/v12.1) (2021-12-30)
 
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v1.1.4...v12)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v2.0.0...v12.1)
+
+**Fixed bugs:**
+
+- \[BUG\] The `other_modified_files` contains all modified files even those that match the file `filter`. [\#293](https://github.com/tj-actions/changed-files/issues/293)
+- \[BUG\] The `only_modified` is evaluated to `true` even when `other_modified_files` is not empty. [\#292](https://github.com/tj-actions/changed-files/issues/292)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
 
 **Merged pull requests:**
 
-- Upgraded to v1.1.4 [\#286](https://github.com/tj-actions/changed-files/pull/286) ([jackton1](https://github.com/jackton1))
+- Fixed regression bug with other\_modified and other\_changed outputs [\#294](https://github.com/tj-actions/changed-files/pull/294) ([jackton1](https://github.com/jackton1))
+- Update reviewdog/action-shellcheck action to v1.13 [\#291](https://github.com/tj-actions/changed-files/pull/291) ([renovate[bot]](https://github.com/apps/renovate))
+- Upgraded to v2.0.0 [\#290](https://github.com/tj-actions/changed-files/pull/290) ([jackton1](https://github.com/jackton1))
+- Updated README.md [\#289](https://github.com/tj-actions/changed-files/pull/289) ([jackton1](https://github.com/jackton1))
 
-## [v1.1.4](https://github.com/tj-actions/changed-files/tree/v1.1.4) (2021-12-14)
+## [v2.0.0](https://github.com/tj-actions/changed-files/tree/v2.0.0) (2021-12-17)
 
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.9...v1.1.4)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v12...v2.0.0)
+
+**Fixed bugs:**
+
+- \[BUG\] Files input doesn't work when running with `act` [\#251](https://github.com/tj-actions/changed-files/issues/251)
+
+**Closed issues:**
+
+- Breaking changes in patch updates? [\#288](https://github.com/tj-actions/changed-files/issues/288)
+
+**Merged pull requests:**
+
+- Upgraded to v12 [\#287](https://github.com/tj-actions/changed-files/pull/287) ([jackton1](https://github.com/jackton1))
+
+## [v12](https://github.com/tj-actions/changed-files/tree/v12) (2021-12-14)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.9...v12)
 
 **Implemented enhancements:**
 
@@ -20,12 +48,9 @@
 
 - \[BUG\] Failed to get change files on pull\_request merge event [\#281](https://github.com/tj-actions/changed-files/issues/281)
 
-**Closed issues:**
-
-- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
-
 **Merged pull requests:**
 
+- Upgraded to v1.1.4 [\#286](https://github.com/tj-actions/changed-files/pull/286) ([jackton1](https://github.com/jackton1))
 - \[PR 2\]: Added support for listing all\_modified\_files. [\#285](https://github.com/tj-actions/changed-files/pull/285) ([jackton1](https://github.com/jackton1))
 - Update peter-evans/create-pull-request action to v3.12.0 [\#284](https://github.com/tj-actions/changed-files/pull/284) ([renovate[bot]](https://github.com/apps/renovate))
 - \[PR 1\]: Renamed all\_modified\_files to all\_changed\_files [\#283](https://github.com/tj-actions/changed-files/pull/283) ([jackton1](https://github.com/jackton1))
@@ -122,7 +147,7 @@
 
 ## [v1.2.1](https://github.com/tj-actions/changed-files/tree/v1.2.1) (2021-11-08)
 
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v1.2.0...v1.2.1)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.5...v1.2.1)
 
 **Implemented enhancements:**
 
@@ -136,13 +161,13 @@
 - Update actions/checkout action to v2.4.0 [\#243](https://github.com/tj-actions/changed-files/pull/243) ([renovate[bot]](https://github.com/apps/renovate))
 - Upgraded to v11.5 [\#241](https://github.com/tj-actions/changed-files/pull/241) ([jackton1](https://github.com/jackton1))
 
-## [v1.2.0](https://github.com/tj-actions/changed-files/tree/v1.2.0) (2021-10-30)
-
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.5...v1.2.0)
-
 ## [v11.5](https://github.com/tj-actions/changed-files/tree/v11.5) (2021-10-30)
 
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.4...v11.5)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v1.2.0...v11.5)
+
+## [v1.2.0](https://github.com/tj-actions/changed-files/tree/v1.2.0) (2021-10-30)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v11.4...v1.2.0)
 
 **Merged pull requests:**
 

README.md

@@ -33,16 +33,6 @@ Retrieve all changed files relative to the default branch (`pull_request*` based
     *   Report on files that have at least one change.
     *   [Regex pattern](https://www.gnu.org/software/grep/manual/grep.html#Regular-Expressions) matching on a subset of files.
 
-
-> NOTE: :warning:
-> * All users referencing the legacy `v1.x.x` version, are required to switch to `v*` or `v*.*` versioning semantic.
-> * A breaking change was introduced in `v1.x.x` legacy version, which is no longer supported and future updates would only be made to `v*` or `v*.*`.
-
-|  Legacy version     |   Recommended version |
-|---------------------|-----------------------|
-|  `v1.x.x`           |    `v12`          |
-
-
 ## Usage
 
 > NOTE: :warning:
@@ -73,7 +63,7 @@ jobs:
 
       - name: Get changed files
         id: changed-files
-        uses: tj-actions/changed-files@v12
+        uses: tj-actions/changed-files@v12.1
 
       - name: List all changed files
         run: |
@@ -150,11 +140,11 @@ Support this project with a :star:
 
       - name: Get changed files using defaults
         id: changed-files
-        uses: tj-actions/changed-files@v12
+        uses: tj-actions/changed-files@v12.1
 
       - name: Get changed files using a comma separator
         id: changed-files-comma
-        uses: tj-actions/changed-files@v12
+        uses: tj-actions/changed-files@v12.1
         with:
           separator: ","
 
@@ -165,18 +155,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@v12
+        uses: tj-actions/changed-files@v12.1
         with:
           files: |
             my-file.txt
@@ -189,12 +179,12 @@ Support this project with a :star:
             ^(mynewfile|custom)
 
       - 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."
 
@@ -214,14 +204,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@v12
+        uses: tj-actions/changed-files@v12.1
         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@v12
+        uses: tj-actions/changed-files@v12.1
         with:
           files_from_source_file: |
             test/changed-files-list.txt
@@ -230,13 +220,13 @@ Support this project with a :star:
 
       - name: Use a different commit SHA
         id: changed-files-custom-sha
-        uses: tj-actions/changed-files@v12
+        uses: tj-actions/changed-files@v12.1
         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@v12
+        uses: tj-actions/changed-files@v12.1
         with:
           base_sha: "2096ed0"
           
@@ -248,7 +238,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@v12
+        uses: tj-actions/changed-files@v12.1
         with:
           path: dir1
 
@@ -260,9 +250,9 @@ 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@v12
+        uses: tj-actions/changed-files@v12.1
         with:
-          since_last_remote_commit: 'true'
+          since_last_remote_commit: "true"
 
 ```
 
@@ -277,7 +267,7 @@ Support this project with a :star:
 
       - name: Get changed files
         id: changed-files
-        uses: tj-actions/changed-files@v12
+        uses: tj-actions/changed-files@v12.1
 
       - name: Pre-commit
         uses: pre-commit/action@v2.0.0
@@ -340,7 +330,3 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
 <!-- ALL-CONTRIBUTORS-LIST:END -->
 
 This project follows the [all-contributors](https://github.com/all-contributors/all-contributors) specification. Contributions of any kind welcome!
-
-[buymeacoffee]: https://www.buymeacoffee.com/jackton1
-
-[buymeacoffee-shield]: https://www.buymeacoffee.com/assets/img/custom_images/orange_img.png

entrypoint.sh

@@ -126,14 +126,14 @@ else
   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 "${UNIQUE_ALL_CHANGED}" | awk '{gsub(/\|/,"\n"); print $0;}' | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
-
   if [[ -n "${UNIQUE_ALL_CHANGED}" ]]; then
     echo "Matching changed files: ${UNIQUE_ALL_CHANGED}"
     echo "::set-output name=any_changed::true"
   else
     echo "::set-output name=any_changed::false"
   fi
+  
+  OTHER_CHANGED=""
 
   if [[ -n $ALL_OTHER_CHANGED ]]; then
     if [[ -n "$UNIQUE_ALL_CHANGED" ]]; then
@@ -156,14 +156,14 @@ else
   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 "${UNIQUE_ALL_MODIFIED}" | awk '{gsub(/\|/,"\n"); print $0;}' | awk -v d="$INPUT_SEPARATOR" '{s=(NR==1?s:s d)$0}END{print s}')
-
   if [[ -n "${UNIQUE_ALL_MODIFIED}" ]]; then
     echo "Matching modified files: ${UNIQUE_ALL_MODIFIED}"
     echo "::set-output name=any_modified::true"
   else
     echo "::set-output name=any_modified::false"
   fi
+  
+  OTHER_MODIFIED=""
 
   if [[ -n $ALL_OTHER_MODIFIED ]]; then
     if [[ -n "$UNIQUE_ALL_MODIFIED" ]]; then
@@ -192,10 +192,12 @@ else
   else
     echo "::set-output name=any_deleted::false"
   fi
+  
+  OTHER_DELETED=""
 
   if [[ -n $ALL_OTHER_DELETED ]]; then
     if [[ -n "$UNIQUE_ALL_DELETED" ]]; then
-      OTHER_DELETED=$(echo "${ALL_OTHER_DELETED}|${UNIQUE_ALL_DELETED}"  | awk '{gsub(/\|/,"\n"); print $0;}' | sort | uniq -u | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
+      OTHER_DELETED=$(echo "${ALL_OTHER_DELETED}|${UNIQUE_ALL_DELETED}" | awk '{gsub(/\|/,"\n"); print $0;}' | sort | uniq -u | awk -v d="|" '{s=(NR==1?s:s d)$0}END{print s}')
     else
       OTHER_DELETED=$ALL_OTHER_DELETED
     fi