Add warning message when the head sha is empty (#56)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

.github/workflows/sync-release-version.yml

@@ -12,7 +12,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Sync release version.
-        uses: tj-actions/sync-release-version@v8.5
+        uses: tj-actions/sync-release-version@v8.6
         id: sync-release-version
         with:
           pattern: '${{ github.repository }}@'

HISTORY.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [v5.2](https://github.com/tj-actions/changed-files/tree/v5.2) (2021-05-06)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v5.1...v5.2)
+
+**Implemented enhancements:**
+
+- \[shellcheck\]: Resolve SC2001 [\#42](https://github.com/tj-actions/changed-files/issues/42)
+- \[shellcheck\]: Resolve SC2128 [\#41](https://github.com/tj-actions/changed-files/issues/41)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
+
+**Merged pull requests:**
+
+- Update tj-actions/github-changelog-generator action to v1.8 [\#52](https://github.com/tj-actions/changed-files/pull/52) ([renovate[bot]](https://github.com/apps/renovate))
+- Upgraded to v5.1 [\#51](https://github.com/tj-actions/changed-files/pull/51) ([jackton1](https://github.com/jackton1))
+
 ## [v5.1](https://github.com/tj-actions/changed-files/tree/v5.1) (2021-05-01)
 
 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v5...v5.1)
@@ -47,7 +65,6 @@
 
 **Closed issues:**
 
-- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
 - Add support for watching a subset of files. [\#20](https://github.com/tj-actions/changed-files/issues/20)
 
 **Merged pull requests:**

README.md

@@ -20,6 +20,9 @@ Get all modified files relative to the default branch (`pull_request*` events) o
 
 > NOTE: :warning:
 > * For `push` events to work 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.
 
 
 
@@ -45,7 +48,7 @@ jobs:
       
       - name: Get changed files
         id: changed-files
-        uses: tj-actions/changed-files@v5.1
+        uses: tj-actions/changed-files@v5.2
       
       - name: List all modified files
         run: |
@@ -71,8 +74,8 @@ jobs:
 
 |   Output             |    type      |  example                       |         description                      |
 |:-------------------:|:------------:|:------------------------------:|:----------------------------------------:|
-| all_changed          |  `string`     |     `true` OR `false`             |  Returns `true` only when the filenames provided using `files` input have all changed |
-| any_changed          |  `string`     |     `true` OR `false`             |  Returns `true` when any of the filenames provided using `files` input have changed |
+| all_changed          |  `string`     |     `true` OR `false`             |  Returns `true` only when the filenames provided using the `files` input have all changed |
+| any_changed          |  `string`     |     `true` OR `false`             |  Returns `true` when any of the filenames provided using the `files` input has changed |
 | all_modified_files   |  `string`    |    `'new.txt other.png ...'`     |  Select all modified files <br /> *i.e a combination of all added, <br />copied and modified files (ACM).*  |
 | all_changed_files   |  `string`    |    `'new.txt other.png ...'`     |  Select all paths (*) <br /> *i.e a combination of all options below.*  |
 | added_files         |  `string`    |    `'new.txt other.png ...'`    |  Select only files that are Added (A)    |
@@ -89,8 +92,9 @@ jobs:
 
 |   Input       |    type    |  required      |  default                      |  description  |
 |:-------------:|:-----------:|:-------------:|:----------------------------:|:-------------:|
+| token         |  `string`   |    `false`    | `${{ github.token }}` | [GITHUB_TOKEN](https://docs.github.com/en/free-pro-team@latest/actions/reference/authentication-in-a-workflow#using-the-github_token-in-a-workflow) <br /> or a repo scoped <br /> [Personal Access Token](https://docs.github.com/en/free-pro-team@latest/github/authenticating-to-github/creating-a-personal-access-token)              |
 | separator         |  `string`   |    `true` |                          `' '` |  Output string separator   |
-| files         |  `string OR string[]`   |    `false` |                           |  Restricted list of specific files to watch for changes |
+| files         |  `string` OR `string[]`   |    `false` |                           |  Restricted list or string of specific <br> files or filename to watch for changes |
 
 
 ## Example
@@ -99,13 +103,14 @@ jobs:
 ...
     steps:
       - uses: actions/checkout@v2
+
       - name: Get changed files using defaults
         id: changed-files
-        uses: tj-actions/changed-files@v5.1
+        uses: tj-actions/changed-files@v5.2
       
       - name: Get changed files using a comma separator
         id: changed-files-comma
-        uses: tj-actions/changed-files@v5.1
+        uses: tj-actions/changed-files@v5.2
         with:
           separator: ","
 
@@ -127,7 +132,7 @@ jobs:
 
       - name: Get specific changed files
         id: changed-files-specific
-        uses: tj-actions/changed-files@v5.1
+        uses: tj-actions/changed-files@v5.2
         with:
           files: |
             my-file.txt
@@ -157,11 +162,11 @@ jobs:
     steps:
       - uses: actions/checkout@v2
         with:
-          fetch-depth: 0  # otherwise, you will fail to push refs to dest repo
+          fetch-depth: 0
       
       - name: Get changed files
         id: changed-files
-        uses: tj-actions/changed-files@v5.1
+        uses: tj-actions/changed-files@v5.2
 
       - name: Pre-commit
         uses: pre-commit/action@v2.0.0

action.yml

@@ -72,57 +72,64 @@ runs:
           git fetch --depth=1 origin "${TARGET_BRANCH}":"${TARGET_BRANCH}"
           HEAD_SHA=$(git rev-parse "${TARGET_BRANCH}" || true)
         fi
-        echo "Using head sha: $HEAD_SHA..."
-        if [[ -z "$INPUT_FILES" ]]; then
-          echo "Getting diff..."
-          ADDED=$(git diff --diff-filter=A --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          COPIED=$(git diff --diff-filter=C --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          DELETED=$(git diff --diff-filter=D --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          MODIFIED=$(git diff --diff-filter=M --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          RENAMED=$(git diff --diff-filter=R --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          CHANGED=$(git diff --diff-filter=T --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          UNMERGED=$(git diff --diff-filter=U --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          UNKNOWN=$(git diff --diff-filter=X --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          ALL_CHANGED=$(git diff --diff-filter="*ACDMRTUX" --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
-          ALL_MODIFIED_FILES=$(git diff --diff-filter="ACM" --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+        
+        if [[ -z $HEAD_SHA ]]; then
+          echo "::warning::Unable to determine the head sha: $HEAD_SHA."
+          echo "::warning::You seem to be misssing `fetch-depth: 0` or `fetch-depth: 2`"
         else
-          ADDED_ARRAY=()
-          COPIED_ARRAY=()
-          DELETED_ARRAY=()
-          MODIFIED_ARRAY=()
-          RENAMED_ARRAY=()
-          CHANGED_ARRAY=()
-          UNMERGED_ARRAY=()
-          UNKNOWN_ARRAY=()
-          ALL_CHANGED_ARRAY=()
-          ALL_MODIFIED_FILES_ARRAY=()
-          for path in ${INPUT_FILES}
-          do
-            echo "Checking for file changes: \"${path}\"..."
-            IFS=" "
-            ADDED_ARRAY+=("$(git diff --diff-filter=A --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            COPIED_ARRAY+=("$(git diff --diff-filter=C --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            DELETED_ARRAY+=("$(git diff --diff-filter=D --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            MODIFIED_ARRAY+=("$(git diff --diff-filter=M --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            RENAMED_ARRAY+=("$(git diff --diff-filter=R --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            CHANGED_ARRAY+=("$(git diff --diff-filter=T --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            UNMERGED_ARRAY+=("$(git diff --diff-filter=U --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            UNKNOWN_ARRAY+=("$(git diff --diff-filter=X --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            ALL_CHANGED_ARRAY+=("$(git diff --diff-filter="*ACDMRTUX" --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-            ALL_MODIFIED_FILES_ARRAY+=("$(git diff --diff-filter="ACM" --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
-          done
+          echo "Using head sha: $HEAD_SHA..."
 
-          ADDED=$(echo "${ADDED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          COPIED=$(echo "${COPIED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          DELETED=$(echo "${DELETED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          MODIFIED=$(echo "${MODIFIED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          RENAMED=$(echo "${RENAMED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          CHANGED=$(echo "${CHANGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          UNMERGED=$(echo "${UNMERGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          UNKNOWN=$(echo "${UNKNOWN_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          ALL_CHANGED=$(echo "${ALL_CHANGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
-          ALL_MODIFIED_FILES=$(echo "${ALL_MODIFIED_FILES_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+          if [[ -z "$INPUT_FILES" ]]; then
+            echo "Getting diff..."
+            ADDED=$(git diff --diff-filter=A --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            COPIED=$(git diff --diff-filter=C --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            DELETED=$(git diff --diff-filter=D --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            MODIFIED=$(git diff --diff-filter=M --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            RENAMED=$(git diff --diff-filter=R --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            CHANGED=$(git diff --diff-filter=T --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            UNMERGED=$(git diff --diff-filter=U --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            UNKNOWN=$(git diff --diff-filter=X --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            ALL_CHANGED=$(git diff --diff-filter="*ACDMRTUX" --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+            ALL_MODIFIED_FILES=$(git diff --diff-filter="ACM" --name-only "$HEAD_SHA" | tr "\n" "$INPUT_SEPARATOR" | sed -E "s/($INPUT_SEPARATOR)$//")
+          else
+            ADDED_ARRAY=()
+            COPIED_ARRAY=()
+            DELETED_ARRAY=()
+            MODIFIED_ARRAY=()
+            RENAMED_ARRAY=()
+            CHANGED_ARRAY=()
+            UNMERGED_ARRAY=()
+            UNKNOWN_ARRAY=()
+            ALL_CHANGED_ARRAY=()
+            ALL_MODIFIED_FILES_ARRAY=()
 
+            for path in ${INPUT_FILES}
+            do
+              echo "Checking for file changes: \"${path}\"..."
+              IFS=" "
+              ADDED_ARRAY+=("$(git diff --diff-filter=A --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              COPIED_ARRAY+=("$(git diff --diff-filter=C --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              DELETED_ARRAY+=("$(git diff --diff-filter=D --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              MODIFIED_ARRAY+=("$(git diff --diff-filter=M --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              RENAMED_ARRAY+=("$(git diff --diff-filter=R --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              CHANGED_ARRAY+=("$(git diff --diff-filter=T --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              UNMERGED_ARRAY+=("$(git diff --diff-filter=U --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              UNKNOWN_ARRAY+=("$(git diff --diff-filter=X --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              ALL_CHANGED_ARRAY+=("$(git diff --diff-filter="*ACDMRTUX" --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+              ALL_MODIFIED_FILES_ARRAY+=("$(git diff --diff-filter="ACM" --name-only "$HEAD_SHA" | grep -E "(${path})" | xargs printf "%s$INPUT_SEPARATOR" || true)")
+            done
+
+            ADDED=$(echo "${ADDED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            COPIED=$(echo "${COPIED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            DELETED=$(echo "${DELETED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            MODIFIED=$(echo "${MODIFIED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            RENAMED=$(echo "${RENAMED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            CHANGED=$(echo "${CHANGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            UNMERGED=$(echo "${UNMERGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            UNKNOWN=$(echo "${UNKNOWN_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            ALL_CHANGED=$(echo "${ALL_CHANGED_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+            ALL_MODIFIED_FILES=$(echo "${ALL_MODIFIED_FILES_ARRAY[@]}" | sed -E "s/($INPUT_SEPARATOR)$//" | xargs)
+          fi
         fi
 
         echo "Added files: $ADDED"
@@ -165,6 +172,7 @@ runs:
             echo "::set-output name=any_changed::false"
           fi
         fi
+
         echo "::set-output name=added_files::$ADDED"
         echo "::set-output name=copied_files::$COPIED"
         echo "::set-output name=deleted_files::$DELETED"