fix: bug returning an error for the first repo commit (#554 )

chore: update README.md (#552 )
* chore: update README.md * Update README.md
2022-07-21 20:33:15 -04:00 · 2022-07-21 18:37:16 -04:00 · 2022-07-21 18:36:35 -04:00 · 2022-07-18 17:34:45 -04:00 · 2022-07-18 16:34:41 -04:00 · 2022-07-06 21:36:59 -04:00
6 changed files with 178 additions and 47 deletions
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -114,6 +114,15 @@
      "contributions": [
        "doc"
      ]
+    },
+    {
+      "login": "JoeOvo",
+      "name": "Joe Moggridge",
+      "avatar_url": "https://avatars.githubusercontent.com/u/100686542?v=4",
+      "profile": "https://github.com/JoeOvo",
+      "contributions": [
+        "doc"
+      ]
    }
  ],
  "contributorsPerLine": 7,
--- a/.github/workflows/sync-release-version.yml
+++ b/.github/workflows/sync-release-version.yml
@@ -19,7 +19,7 @@ jobs:
          paths: |
            README.md
      - name: Generate CHANGELOG
-        uses: tj-actions/github-changelog-generator@v1.13
+        uses: tj-actions/github-changelog-generator@v1.14
        with:
          output: 'HISTORY.md'
      - name: Create Pull Request
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -8,6 +8,9 @@ on:
  pull_request:
    branches:
      - main
+  pull_request_review:
+    types:
+      - submitted

 jobs:
  shellcheck:
@@ -116,6 +119,29 @@ jobs:
        shell:
          bash

+  test-similar-base-and-commit-sha:
+    name: Test changed-files similar base and commit sha
+    runs-on: ${{ matrix.platform }}
+    strategy:
+      fail-fast: false
+      max-parallel: 4
+      matrix:
+        platform: [ubuntu-latest, windows-latest, macos-latest, macos-11, ubuntu-18.04, windows-2022]
+
+    steps:
+      - name: Checkout to branch
+        uses: actions/checkout@v3
+
+      - name: Run changed-files with a single commit history
+        id: changed-files
+        uses: ./
+
+      - name: Show output
+        run: |
+          echo '${{ toJSON(steps.changed-files.outputs) }}'
+        shell:
+          bash
+
  test-non-existent-base-sha:
    name: Test changed-files non existent base sha
    runs-on: ${{ matrix.platform }}
@@ -141,6 +167,12 @@ jobs:
          echo "${{ toJSON(steps.changed-files.outputs) }}"
        shell:
          bash
+      
+      - name: Exit with 1 if no error is raised
+        if: steps.changed-files.outcome != 'failure'
+        run: |
+          echo "Expected: (failure) got ${{ steps.changed-files.outcome }}"
+          exit 1

      - name: Run changed-files-specific with non existent base sha
        id: changed-files-specific
@@ -155,6 +187,12 @@ jobs:
          echo "${{ toJSON(steps.changed-files-specific.outputs) }}"
        shell:
          bash
+      
+      - name: Exit with 1 if no error is raised
+        if: steps.changed-files-specific.outcome != 'failure'
+        run: |
+          echo "Expected: (failure) got ${{ steps.changed-files-specific.outcome }}"
+          exit 1

  test-non-existent-sha:
    name: Test changed-files non existent sha
@@ -181,6 +219,12 @@ jobs:
          echo "${{ toJSON(steps.changed-files.outputs) }}"
        shell:
          bash
+      
+      - name: Exit with 1 if no error is raised
+        if: steps.changed-files.outcome != 'failure'
+        run: |
+          echo "Expected: (failure) got ${{ steps.changed-files.outcome }}"
+          exit 1

      - name: Run changed-files-specific with non existent sha
        id: changed-files-specific
@@ -195,6 +239,12 @@ jobs:
          echo "${{ toJSON(steps.changed-files-specific.outputs) }}"
        shell:
          bash
+      
+      - name: Exit with 1 if no error is raised
+        if: steps.changed-files-specific.outcome != 'failure'
+        run: |
+          echo "Expected: (failure) got ${{ steps.changed-files-specific.outcome }}"
+          exit 1

  test:
    name: Test changed-files
--- a/HISTORY.md
+++ b/HISTORY.md
@@ -1,5 +1,61 @@
 # Changelog

+## [Unreleased](https://github.com/tj-actions/changed-files/tree/HEAD)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v23.2...HEAD)
+
+**Fixed bugs:**
+
+- \[BUG\] PREVIOUS\_SHA & CURRENT\_SHA is the same when re-running a workflow, causing changed-files to be empty [\#547](https://github.com/tj-actions/changed-files/issues/547)
+
+## [v23.2](https://github.com/tj-actions/changed-files/tree/v23.2) (2022-07-18)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v23.1...v23.2)
+
+**Fixed bugs:**
+
+- \[BUG\] Repo name changed, workflows are broken [\#542](https://github.com/tj-actions/changed-files/issues/542)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
+
+**Merged pull requests:**
+
+- feat: fix bug with similar commit hashes. [\#549](https://github.com/tj-actions/changed-files/pull/549) ([jackton1](https://github.com/jackton1))
+- chore: update readme [\#544](https://github.com/tj-actions/changed-files/pull/544) ([jackton1](https://github.com/jackton1))
+- chore\(deps\): update tj-actions/github-changelog-generator action to v1.14 [\#541](https://github.com/tj-actions/changed-files/pull/541) ([renovate[bot]](https://github.com/apps/renovate))
+- Updated README.md [\#540](https://github.com/tj-actions/changed-files/pull/540) ([jackton1](https://github.com/jackton1))
+- chore: update test [\#539](https://github.com/tj-actions/changed-files/pull/539) ([jackton1](https://github.com/jackton1))
+- chore: update README.md [\#538](https://github.com/tj-actions/changed-files/pull/538) ([jackton1](https://github.com/jackton1))
+- docs: add JoeOvo as a contributor for doc [\#537](https://github.com/tj-actions/changed-files/pull/537) ([allcontributors[bot]](https://github.com/apps/allcontributors))
+- Update README.md [\#536](https://github.com/tj-actions/changed-files/pull/536) ([JoeOvo](https://github.com/JoeOvo))
+- Upgraded to v23.1 [\#535](https://github.com/tj-actions/changed-files/pull/535) ([jackton1](https://github.com/jackton1))
+
+## [v23.1](https://github.com/tj-actions/changed-files/tree/v23.1) (2022-06-24)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v23...v23.1)
+
+**Implemented enhancements:**
+
+- \[Feature\]  implement an outputs for matrix compatible jobs [\#530](https://github.com/tj-actions/changed-files/issues/530)
+
+**Fixed bugs:**
+
+- \[BUG\] Misleading error if git command is missing [\#532](https://github.com/tj-actions/changed-files/issues/532)
+
+**Merged pull requests:**
+
+- chore: removed unused code [\#534](https://github.com/tj-actions/changed-files/pull/534) ([jackton1](https://github.com/jackton1))
+- chore: improve error handling [\#533](https://github.com/tj-actions/changed-files/pull/533) ([jackton1](https://github.com/jackton1))
+- chore\(deps\): update tj-actions/verify-changed-files action to v10 [\#531](https://github.com/tj-actions/changed-files/pull/531) ([renovate[bot]](https://github.com/apps/renovate))
+- Updated README.md [\#528](https://github.com/tj-actions/changed-files/pull/528) ([jackton1](https://github.com/jackton1))
+- chore\(deps\): update codacy/codacy-analysis-cli-action action to v4.1.0 [\#527](https://github.com/tj-actions/changed-files/pull/527) ([renovate[bot]](https://github.com/apps/renovate))
+- docs: add deronnax as a contributor for doc [\#526](https://github.com/tj-actions/changed-files/pull/526) ([allcontributors[bot]](https://github.com/apps/allcontributors))
+- fix mispellings [\#525](https://github.com/tj-actions/changed-files/pull/525) ([deronnax](https://github.com/deronnax))
+- chore: reformat manual-test.yml [\#524](https://github.com/tj-actions/changed-files/pull/524) ([jackton1](https://github.com/jackton1))
+- Upgraded to v23 [\#523](https://github.com/tj-actions/changed-files/pull/523) ([jackton1](https://github.com/jackton1))
+
 ## [v23](https://github.com/tj-actions/changed-files/tree/v23) (2022-06-12)

 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v22.2...v23)
@@ -7,16 +63,11 @@
 **Implemented enhancements:**

 - \[Feature\] Get distinct changed folders [\#504](https://github.com/tj-actions/changed-files/issues/504)
- \[Feature\] Rename `files` -\> `paths` [\#125](https://github.com/tj-actions/changed-files/issues/125)

 **Fixed bugs:**

 - \[BUG\] Action does not work when running with `act` anymore [\#518](https://github.com/tj-actions/changed-files/issues/518)

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

 - feat: add support for returning directory names [\#522](https://github.com/tj-actions/changed-files/pull/522) ([jackton1](https://github.com/jackton1))
@@ -1157,7 +1208,7 @@

 ## [v7](https://github.com/tj-actions/changed-files/tree/v7) (2021-06-09)

-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6...v7)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6.3...v7)

 **Closed issues:**

@@ -1173,13 +1224,13 @@
 - Update tj-actions/sync-release-version action to v8.7 [\#86](https://github.com/tj-actions/changed-files/pull/86) ([renovate[bot]](https://github.com/apps/renovate))
 - Upgraded to v6.3 [\#85](https://github.com/tj-actions/changed-files/pull/85) ([jackton1](https://github.com/jackton1))

-## [v6](https://github.com/tj-actions/changed-files/tree/v6) (2021-05-26)
-
-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6.3...v6)
-
 ## [v6.3](https://github.com/tj-actions/changed-files/tree/v6.3) (2021-05-26)

-[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6.2...v6.3)
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6...v6.3)
+
+## [v6](https://github.com/tj-actions/changed-files/tree/v6) (2021-05-26)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v6.2...v6)

 **Merged pull requests:**

--- 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-12-orange.svg?style=flat-square)](#contributors-)
+[![All Contributors](https://img.shields.io/badge/all_contributors-13-orange.svg?style=flat-square)](#contributors-)

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

@@ -42,7 +42,7 @@ Retrieve all changed files and directories relative to the target branch (`pull_
 > NOTE: :warning:
 >
 > *   **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.
+> *   When using `persist-credentials: false` with `actions/checkout@v3` you'll need to specify a `token` using the `token` input.
 > *   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).

@@ -68,7 +68,7 @@ jobs:

      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2

      - name: List all changed files
        run: |
@@ -102,19 +102,19 @@ Support this project with a :star:

 |               Output               |   type   |                            example                             |                                                                                                      description                                                                                                      |
 |:----------------------------------:|:--------:|:--------------------------------------------------------------:|:---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------:|
-|            any\_changed            | `string` |                       `true` OR `false`                        |         Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has changed. <br /> i.e. *using a combination of all added, <br />copied, modified and renamed files (ACMR)*          |
-|           only\_changed            | `string` |                       `true` OR `false`                        |                                                           Returns `true` when only <br /> files provided using <br /> the `files` input has changed. (ACMR)                                                           |
+|            any\_changed            | `string` |                       `true` OR `false`                        |         Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has changed. If no `files` have been specified,<br> an empty string `''` is returned. <br /> i.e. *using a combination of all added, <br />copied, modified and renamed files (ACMR)*          |
+|           only\_changed            | `string` |                       `true` OR `false`                        |                                                           Returns `true` when only <br /> files provided using <br /> the `files` input has changed. If no `files` have been specified,<br> an empty string `''` is returned. (ACMR)                                                           |
 |       other\_changed\_files        | `string` |                `'new.txt path/to/file.png ...'`                |                                 Select all other changed files <br/> not listed in the files input <br /> i.e. *a  combination of all added, <br /> copied and modified files (ACMR)*                                 |
-|           any\_modified            | `string` |                       `true` OR `false`                        | Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has been modified. <br /> i.e. *using a combination of all added, <br />copied, modified, renamed, and deleted files (ACMRD)* |
-|           only\_modified           | `string` |                       `true` OR `false`                        |                                                       Returns `true` when only <br /> files provided using <br /> the `files` input has been modified. (ACMRD)                                                        |
+|           any\_modified            | `string` |                       `true` OR `false`                        | Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has been modified. If no `files` have been specified,<br> an empty string `''` is returned. <br /> i.e. *using a combination of all added, <br />copied, modified, renamed, and deleted files (ACMRD)* |
+|           only\_modified           | `string` |                       `true` OR `false`                        |                                                       Returns `true` when only <br /> files provided using <br /> the `files` input has been modified. If no `files` have been specified,<br> an empty string `''` is returned.(ACMRD)                                                        |
 |       other\_modified\_files       | `string` |                `'new.txt path/to/file.png ...'`                |                           Select all other modified files <br/> not listed in the files input <br /> i.e. *a  combination of all added, <br /> copied, modified, and deleted files (ACMRD)*                           |
-|            any\_deleted            | `string` |                       `true` OR `false`                        |                                                     Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has been deleted. (D)                                                     |
-|           only\_deleted            | `string` |                       `true` OR `false`                        |                                                          Returns `true` when only <br /> files provided using <br /> the `files` input has been deleted. (D)                                                          |
+|            any\_deleted            | `string` |                       `true` OR `false`                        |                                                     Returns `true` when any <br /> of the filenames provided using <br /> the `files` input has been deleted. If no `files` have been specified,<br> an empty string `''` is returned. (D)                                                     |
+|           only\_deleted            | `string` |                       `true` OR `false`                        |                                                          Returns `true` when only <br /> files provided using <br /> the `files` input has been deleted. If no `files` have been specified,<br> an empty string `''` is returned. (D)                                                          |
 |       other\_deleted\_files        | `string` |                `'new.txt path/to/file.png ...'`                |                                               Select all other deleted files <br/> not listed in the files input <br /> i.e. *a  combination of all deleted files (D)*                                                |
 |        all\_changed\_files         | `string` |                `'new.txt path/to/file.png ...'`                |                                                  Select all changed files <br /> i.e. *a combination of all added, <br />copied, modified and renamed files (ACMR)*                                                   |
 |        all\_modified\_files        | `string` |                `'new.txt path/to/file.png ...'`                |                                             Select all changed files <br /> i.e. *a combination of all added, <br />copied, modified, renamed and deleted files (ACMRD)*                                              |
 | all\_changed\_and\_modified\_files | `string` |                `'new.txt path/to/file.png ...'`                |                                                                Select all changed <br /> and modified files <br /> i.e. *a combination of (ACMRDTUX)*                                                                 |
-|   all\_old\_new\_renamed\_files    | `string` | `'old name.txt,new name.txt old name 2.txt,new name 2.txt...'` |                                                                        Select only files that are Renamed and list their old and new names (R)                                                                        |
+|   all\_old\_new\_renamed\_files    | `string` | `'old name.txt,new name.txt old name 2.txt,new name 2.txt...'` |                                                                        Select only files that are Renamed and list their old and new names. <br> NOTE: This requires setting `include_all_old_new_renamed_files` to `true` (R)                                                                        |
 |            added\_files            | `string` |                `'new.txt path/to/file.png ...'`                |                                                                                         Select only files that are Added (A)                                                                                          |
 |           copied\_files            | `string` |                `'new.txt path/to/file.png ...'`                |                                                                                         Select only files that are Copied (C)                                                                                         |
 |           deleted\_files           | `string` |                `'new.txt path/to/file.png ...'`                |                                                                                        Select only files that are Deleted (D)                                                                                         |
@@ -130,7 +130,7 @@ Support this project with a :star:
 |:--------------------------------------:|:----------------------:|:--------:|:---------------------:|:-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------:|
 |                 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`        | `false`  |         `' '`         |                                                                                                                                              Split character for output strings                                                                                                                                               |
-| include\_all\_old\_new\_renamed\_files |       `boolean`        | `false`  |        `false`        |                                                                                 Include `all_old_new_renamed_files` output. Note this can generate a large debug output See: [#501](https://github.com/tj-actions/changed-files/issues/501).                                                                                  |
+| include\_all\_old\_new\_renamed\_files |       `boolean`        | `false`  |        `false`        |                                                                                 Include `all_old_new_renamed_files` output. Note this can generate a large output See: [#501](https://github.com/tj-actions/changed-files/issues/501).                                                                                  |
 |          old\_new\_separator           |        `string`        | `false`  |         `','`         |                                                                                                                                        Split character for old and new filename pairs                                                                                                                                         |
 |       old\_new\_files\_separator       |        `string`        | `false`  |         `' '`         |                                                                                                                                    Split character for multiple old and new filename pairs                                                                                                                                    |
 |                 files                  | `string` OR `string[]` | `false`  |                       |                                                                                                             Check for changes  <br> using only these <br> list of file(s) <br> (Defaults to the <br> entire repo)                                                                                                             |
@@ -142,7 +142,7 @@ Support this project with a :star:
 |                  sha                   |        `string`        |  `true`  |  `${{ github.sha }}`  |                                                                                                                           Specify a different <br> commit SHA <br> used for <br> comparing changes                                                                                                                            |
 |               base\_sha                |        `string`        | `false`  |                       |                                                                                                                         Specify a different <br> base commit SHA <br> used for <br> comparing changes                                                                                                                         |
 |                  path                  |        `string`        | `false`  |         `'.'`         |                                                                                                                              Relative path under <br> `GITHUB_WORKSPACE` <br> to the repository                                                                                                                               |
-|      since\_last\_remote\_commit       |       `boolean`        | `false`  |        `false`        |       Use the last commit on the remote <br> branch as the `base_sha` <br> (Defaults to the last commit <br> on the target branch for Pull requests <br> or the previous commit <br> on the current branch <br> for push events). <br /> NOTE: This requires <br /> `fetch-depth: 0` <br /> with `actions/checkout@v2`        |
+|      since\_last\_remote\_commit       |       `boolean`        | `false`  |        `false`        |       Use the last commit on the remote <br> branch as the `base_sha` <br> (Defaults to the last commit <br> on the target branch for Pull requests <br> or the previous commit <br> on the current branch <br> for push events). <br /> NOTE: This requires <br /> `fetch-depth: 0` <br /> with `actions/checkout@v3`        |
 |            use\_fork\_point            |       `boolean`        | `false`  |        `false`        |           Finds best common ancestor <br /> between two commits <br /> to use in a three-way merge <br /> as the `base_sha` <br /> See: [git merge-base](https://git-scm.com/docs/git-merge-base#Documentation/git-merge-base.txt---fork-point). <br> NOTE: This pulls the entire commit history of the base branch           |
 |               quotepath                |       `boolean`        | `false`  |        `true`         |                                                                                                                                Output filenames completely verbatim by setting this to `false`                                                                                                                                |
 |             diff\_relative             |       `boolean`        | `false`  |                       |                                                                                                                       Exclude changes outside the current directory and show pathnames relative to it.                                                                                                                        |
@@ -160,11 +160,11 @@ Support this project with a :star:

      - name: Get changed files using defaults
        id: changed-files
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2

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

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

      - name: Get specific changed files
        id: changed-files-specific
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          files: |
            my-file.txt
@@ -227,14 +227,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@v23
+        uses: tj-actions/changed-files@v23.2
        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@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          files_from_source_file: |
            test/changed-files-list.txt
@@ -243,25 +243,25 @@ Support this project with a :star:

      - name: Use a different commit SHA
        id: changed-files-custom-sha
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        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@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          base_sha: "2096ed0"
          
      - name: Checkout into dir1
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
        with:
          fetch-depth: 0
          path: dir1

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

@@ -273,19 +273,19 @@ 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@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          since_last_remote_commit: "true"
      
      - name: Run changed-files using the fork point of a pull request
        id: changed-files-fork-point
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          use_fork_point: "true"
          
      - name: Run changed-files with quotepath disabled
        id: changed-files-quotepath
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          quotepath: "false"
      
@@ -307,7 +307,7 @@ Support this project with a :star:

      - name: Run changed-files with the commit of the last successful test workflow run
        id: changed-files-base-sha-push
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          base_sha: ${{ steps.last_successful_commit_push.outputs.commit_hash }}

@@ -328,13 +328,13 @@ Support this project with a :star:
      - name: Run changed-files with the commit of the last successful test workflow run on main
        if: github.event_name == 'pull_request'
        id: changed-files-base-sha-pull-request
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          base_sha: ${{ steps.last_successful_commit_pull_request.outputs.commit_hash }}

      - name: Run changed-files with dir_names
        id: changed-files-dir-names
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          dir_names: "true"
 ```
@@ -387,7 +387,7 @@ With the switch from using grep's Extended regex to match files to the natively

      - name: Get specific changed files
        id: changed-files-specific
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@v23.2
        with:
          files: |
            *.sh
@@ -440,6 +440,7 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
    <td align="center"><a href="https://fagai.net"><img src="https://avatars.githubusercontent.com/u/1772112?v=4?s=100" width="100px;" alt=""/><br /><sub><b>fagai</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=fagai" title="Documentation">📖</a></td>
    <td align="center"><a href="https://github.com/pkit"><img src="https://avatars.githubusercontent.com/u/805654?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Constantine Peresypkin</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=pkit" title="Code">💻</a></td>
    <td align="center"><a href="https://github.com/deronnax"><img src="https://avatars.githubusercontent.com/u/439279?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Mathieu Dupuy</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=deronnax" title="Documentation">📖</a></td>
+    <td align="center"><a href="https://github.com/JoeOvo"><img src="https://avatars.githubusercontent.com/u/100686542?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Joe Moggridge</b></sub></a><br /><a href="https://github.com/tj-actions/changed-files/commits?author=JoeOvo" title="Documentation">📖</a></td>
  </tr>
 </table>

--- a/diff-sha.sh
+++ b/diff-sha.sh
@@ -2,6 +2,8 @@

 set -eu

+INITIAL_COMMIT="false"
+
 echo "::group::changed-files-diff-sha"

 if [[ -n $INPUT_PATH ]]; then
@@ -25,7 +27,7 @@ fi
 echo "::debug::Getting HEAD SHA..."

 if [[ -z $INPUT_SHA ]]; then
-  CURRENT_SHA=$(git rev-list --no-merges -n 1 HEAD 2>&1) && exit_status=$? || exit_status=$?
+  CURRENT_SHA=$(git rev-list -n 1 "HEAD" 2>&1) && exit_status=$? || exit_status=$?
 else
  CURRENT_SHA=$INPUT_SHA && exit_status=$? || exit_status=$?
 fi
@@ -37,25 +39,31 @@ if [[ $exit_status -ne 0 ]]; then
  git --version
  echo "::error::You seem to be missing 'fetch-depth: 0' or 'fetch-depth: 2'. See https://github.com/tj-actions/changed-files#usage"
  exit 1
+else
+  echo "::debug::Current SHA: $CURRENT_SHA"
 fi

 if [[ -z $GITHUB_BASE_REF ]]; then
  TARGET_BRANCH=${GITHUB_REF/refs\/heads\//}
  CURRENT_BRANCH=$TARGET_BRANCH

-  if [[ -z $INPUT_BASE_SHA ]]; then
-    git fetch --no-tags -u --progress --depth=2 origin "${CURRENT_BRANCH}":"${CURRENT_BRANCH}" &&
-    exit_status=$? || exit_status=$?
+  echo "::debug::GITHUB_BASE_REF unset using $TARGET_BRANCH..."

-    if [[ $(git rev-list --count HEAD) -gt 1 ]]; then
-      PREVIOUS_SHA=$(git rev-list --no-merges -n 1 HEAD^1 2>&1) && exit_status=$? || exit_status=$?
+  if [[ -z $INPUT_BASE_SHA ]]; then
+    if [[ $(git rev-list --count "HEAD") -gt 1 ]]; then
+      PREVIOUS_SHA=$(git rev-parse "@~" 2>&1) && exit_status=$? || exit_status=$?
+      echo "::debug::Previous SHA: $PREVIOUS_SHA"
    else
      PREVIOUS_SHA=$CURRENT_SHA && exit_status=$? || exit_status=$?
+      INITIAL_COMMIT="true"
      echo "::debug::Initial commit detected"
+      echo "::debug::Previous SHA: $PREVIOUS_SHA"
    fi
  else
    PREVIOUS_SHA=$INPUT_BASE_SHA && exit_status=$? || exit_status=$?
    TARGET_BRANCH=$(git name-rev --name-only "$PREVIOUS_SHA" 2>&1) && exit_status=$? || exit_status=$?
+    echo "::debug::Previous SHA: $PREVIOUS_SHA"
+    echo "::debug::Target branch: $TARGET_BRANCH"
  fi

  git rev-parse --quiet --verify "$PREVIOUS_SHA^{commit}" 1>/dev/null 2>&1 && exit_status=$? || exit_status=$?
@@ -69,19 +77,25 @@ else
  TARGET_BRANCH=$GITHUB_BASE_REF
  CURRENT_BRANCH=$GITHUB_HEAD_REF

+  echo "::debug::GITHUB_BASE_REF: $TARGET_BRANCH..."
+
  if [[ -z $INPUT_BASE_SHA ]]; then
    if [[ "$INPUT_USE_FORK_POINT" == "true" ]]; then
      echo "::debug::Getting fork point..."
      git fetch --no-tags -u --progress origin "${TARGET_BRANCH}":"${TARGET_BRANCH}" && exit_status=$? || exit_status=$?
      PREVIOUS_SHA=$(git merge-base --fork-point "${TARGET_BRANCH}" "$(git name-rev --name-only "$CURRENT_SHA")") && exit_status=$? || exit_status=$?
+      echo "::debug::Previous SHA: $PREVIOUS_SHA"
    else
      git fetch --no-tags -u --progress origin --depth=1 "${TARGET_BRANCH}":"${TARGET_BRANCH}" && exit_status=$? || exit_status=$?
-      PREVIOUS_SHA=$(git rev-list --no-merges -n 1 "${TARGET_BRANCH}" 2>&1) && exit_status=$? || exit_status=$?
+      PREVIOUS_SHA=$(git rev-list -n 1 "${TARGET_BRANCH}" 2>&1) && exit_status=$? || exit_status=$?
+      echo "::debug::Previous SHA: $PREVIOUS_SHA"
    fi
  else
    git fetch --no-tags -u --progress origin --depth=1 "$INPUT_BASE_SHA" && exit_status=$? || exit_status=$?
    PREVIOUS_SHA=$INPUT_BASE_SHA
    TARGET_BRANCH=$(git name-rev --name-only "$PREVIOUS_SHA" 2>&1) && exit_status=$? || exit_status=$?
+    echo "::debug::Previous SHA: $PREVIOUS_SHA"
+    echo "::debug::Target branch: $TARGET_BRANCH"
  fi

  echo "::debug::Verifying commit SHA..."
@@ -94,6 +108,12 @@ else
  fi
 fi

+if [[ -n "$PREVIOUS_SHA" && -n "$CURRENT_SHA" && "$PREVIOUS_SHA" == "$CURRENT_SHA" && "$INITIAL_COMMIT" == "false" ]]; then
+  echo "::error::Similar commit hashes detected: previous sha: $PREVIOUS_SHA is equivalent to the current sha: $CURRENT_SHA"
+  echo "::error::You seem to be missing 'fetch-depth: 0' or 'fetch-depth: 2'. See https://github.com/tj-actions/changed-files#usage"
+  exit 1
+fi
+
 echo "::set-output name=target_branch::$TARGET_BRANCH"
 echo "::set-output name=current_branch::$CURRENT_BRANCH"
 echo "::set-output name=previous_sha::$PREVIOUS_SHA"
Author	SHA1	Message	Date
Tonye Jack	6c44eb8294	fix: bug returning an error for the first repo commit (#554 )	2022-07-21 20:33:15 -04:00
Tonye Jack	ce1dc8468b	chore: update README.md (#552 ) * chore: update README.md * Update README.md	2022-07-21 18:37:16 -04:00
Tonye Jack	c07b7603d6	chore: Update README.md (#553 )	2022-07-21 18:36:35 -04:00
Tonye Jack	22dc9741b8	Upgraded from v23.1 -> v23.2 (#551 ) Co-authored-by: jacktony1 <jacktony1@users.noreply.github.com>	2022-07-18 17:34:45 -04:00
Tonye Jack	b2ee165d6b	feat: fix bug with similar commit hashes. (#549 ) * feat: improve error handling when similar commit hashed are detected. * Updated triggered event * Update debug message. * Removed --no-merges restriction. * Update to use qoutes. * updated git fetch.	2022-07-18 16:34:41 -04:00
Tonye Jack	8acdf5acdf	chore: update readme (#544 )	2022-07-06 21:36:59 -04:00
renovate[bot]	fcfb36863a	chore(deps): update tj-actions/github-changelog-generator action to v1.14	2022-07-01 07:40:47 -04:00
Tonye Jack	e2e05ab7f8	chore: update test (#539 ) * chore: update test * Update test.yml * Update test.yml	2022-06-29 22:42:32 -04:00
Tonye Jack	665f0cd35c	Updated README.md (#540 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-06-29 22:42:17 -04:00
Tonye Jack	5eb9d8c131	Update README.md (#538 )	2022-06-28 21:14:35 -04:00
allcontributors[bot]	7d9d8dbed9	docs: add JoeOvo as a contributor for doc (#537 ) * 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-06-28 13:37:33 -04:00
Joe Moggridge	736c95931c	Update README.md (#536 ) * Update README.md Make it extra clear that some of these outputs only work if the `files` parameter has been specified. If `files` has not been specified, then they don't work. * Apply suggestions from code review * Update README.md Co-authored-by: Tonye Jack <jtonye@ymail.com>	2022-06-28 13:37:17 -04:00
Tonye Jack	8f98569b2a	Upgraded from v23 -> v23.1 (#535 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-06-24 08:07:26 -04:00