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.
chore: update readme (#544 )
2022-07-18 16:34:41 -04:00 · 2022-07-06 21:36:59 -04:00 · 2022-07-01 07:40:47 -04:00 · 2022-06-29 22:42:32 -04:00 · 2022-06-29 22:42:17 -04:00 · 2022-06-28 21:14:35 -04:00
10 changed files with 200 additions and 62 deletions
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -105,6 +105,24 @@
      "contributions": [
        "code"
      ]
+    },
+    {
+      "login": "deronnax",
+      "name": "Mathieu Dupuy",
+      "avatar_url": "https://avatars.githubusercontent.com/u/439279?v=4",
+      "profile": "https://github.com/deronnax",
+      "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/codacy-analysis.yml
+++ b/.github/workflows/codacy-analysis.yml
@@ -28,7 +28,7 @@ jobs:

      # Execute Codacy Analysis CLI and generate a SARIF output with the security issues identified during the analysis
      - name: Run Codacy Analysis CLI
-        uses: codacy/codacy-analysis-cli-action@4.0.2
+        uses: codacy/codacy-analysis-cli-action@v4.1.0
        with:
          # Check https://github.com/codacy/codacy-analysis-cli#project-token to get your project token from your Codacy repository
          # You can also omit the token and run the tools that support default configurations
--- a/.github/workflows/manual-test.yml
+++ b/.github/workflows/manual-test.yml
@@ -27,6 +27,7 @@ jobs:
      - name: Show output
        run: |
          echo '${{ toJSON(steps.changed-files.outputs) }}'
+
      - name: Run changed-files with glob filtering
        id: changed-files-glob
        uses: ./
--- 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/.github/workflows/update-readme.yml
+++ b/.github/workflows/update-readme.yml
@@ -17,7 +17,7 @@ jobs:
        uses: tj-actions/remark@v3

      - name: Verify Changed files
-        uses: tj-actions/verify-changed-files@v9
+        uses: tj-actions/verify-changed-files@v10
        id: verify_changed_files
        with:
          files: |
--- a/HISTORY.md
+++ b/HISTORY.md
@@ -1,5 +1,53 @@
 # Changelog

+## [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)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
+
+**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)
+
+**Implemented enhancements:**
+
+- \[Feature\] Get distinct changed folders [\#504](https://github.com/tj-actions/changed-files/issues/504)
+
+**Fixed bugs:**
+
+- \[BUG\] Action does not work when running with `act` anymore [\#518](https://github.com/tj-actions/changed-files/issues/518)
+
+**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))
+- feat: use debug messages for log outputs [\#521](https://github.com/tj-actions/changed-files/pull/521) ([jackton1](https://github.com/jackton1))
+- chore: clean up internal variables. [\#520](https://github.com/tj-actions/changed-files/pull/520) ([jackton1](https://github.com/jackton1))
+- chore\(deps\): update peter-evans/create-pull-request action to v4.0.4 [\#519](https://github.com/tj-actions/changed-files/pull/519) ([renovate[bot]](https://github.com/apps/renovate))
+- Upgraded to v22.2 [\#517](https://github.com/tj-actions/changed-files/pull/517) ([jackton1](https://github.com/jackton1))
+
 ## [v22.2](https://github.com/tj-actions/changed-files/tree/v22.2) (2022-06-02)

 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v22.1...v22.2)
@@ -31,10 +79,6 @@

 - \[Feature\] since\_last\_passing\_remote\_commit  [\#507](https://github.com/tj-actions/changed-files/issues/507)

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

 - Updated README.md [\#511](https://github.com/tj-actions/changed-files/pull/511) ([jackton1](https://github.com/jackton1))
@@ -78,10 +122,6 @@

 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v20...v20.1)

-**Implemented enhancements:**
-
- \[Feature\] Rename `files` -\> `paths` [\#125](https://github.com/tj-actions/changed-files/issues/125)
-
 **Fixed bugs:**

 - \[BUG\] Glob pattern doesn't work for markdown files  [\#492](https://github.com/tj-actions/changed-files/issues/492)
@@ -1140,7 +1180,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:**

@@ -1156,13 +1196,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,13 +7,13 @@

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

-[![All Contributors](https://img.shields.io/badge/all_contributors-11-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 -->

 ## changed-files

-Retrieve all changed files relative to the target branch (`pull_request*` based events) or the last remote commit (`push` based event) returning the **absolute path** to all changed files from the project root.
+Retrieve all changed files and directories relative to the target branch (`pull_request*` based events) or the last remote commit (`push` based event) returning the **absolute path** to all changed files and directories from the project root.

 ## Features

@@ -28,12 +28,12 @@ Retrieve all changed files relative to the target branch (`pull_request*` based
 *   [GitHub-hosted runners](https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners) support
 *   [GitHub Enterprise Server](https://docs.github.com/en/enterprise-server@3.3/admin/github-actions/getting-started-with-github-actions-for-your-enterprise/getting-started-with-github-actions-for-github-enterprise-server) support.
 *   [self-hosted runners](https://docs.github.com/en/actions/hosting-your-own-runners/about-self-hosted-runners) support.
-*   List all files that have changed:
+*   List all files and directories that have changed:
    *   Between the current pull request branch and the last commit on the target branch.
    *   Between the current pull request branch and the fork point on the target 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:
+*   Restrict change detection to a subset of files and directories:
    *   Report on files that have at least one change.
    *   Using [Glob pattern](https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet) matching.

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

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

      - 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)                                                                                                             |
@@ -160,11 +160,11 @@ Support this project with a :star:

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

      - name: Get changed files using a comma separator
        id: changed-files-comma
-        uses: tj-actions/changed-files@v22.2
+        uses: tj-actions/changed-files@v23.1
        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@v22.2
+        uses: tj-actions/changed-files@v23.1
        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@v22.2
+        uses: tj-actions/changed-files@v23.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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          files_from_source_file: |
            test/changed-files-list.txt
@@ -243,13 +243,13 @@ Support this project with a :star:

      - name: Use a different commit SHA
        id: changed-files-custom-sha
-        uses: tj-actions/changed-files@v22.2
+        uses: tj-actions/changed-files@v23.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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          base_sha: "2096ed0"
          
@@ -261,7 +261,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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          path: dir1

@@ -273,25 +273,25 @@ 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@v22.2
+        uses: tj-actions/changed-files@v23.1
        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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          use_fork_point: "true"
          
      - name: Run changed-files with quotepath disabled
        id: changed-files-quotepath
-        uses: tj-actions/changed-files@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          quotepath: "false"
      
      # Run changed-files action using the last successful commit as the base_sha
      # NOTE: This setting overrides the commit sha used by setting since_last_remote_commit to true.
-      # It is recommended to use either solutions that works for your use case.
+      # It is recommended to use either solution that works for your use case.
      
      # Push event based workflows
      - name: Get branch name
@@ -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@v22.2
+        uses: tj-actions/changed-files@v23.1
        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@v22.2
+        uses: tj-actions/changed-files@v23.1
        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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          dir_names: "true"
 ```
@@ -363,7 +363,7 @@ This project follows a `v(major).(patch)` versioning scheme with the exception o

 ## Migration guide

-With the switch from using grep's Extended regex to match files to the natively supported workflow glob pattern matching snytax introduced in [v13](https://github.com/tj-actions/changed-files/releases/tag/v13) you'll need to modify patterns used to match `files`.
+With the switch from using grep's Extended regex to match files to the natively supported workflow glob pattern matching syntax introduced in [v13](https://github.com/tj-actions/changed-files/releases/tag/v13) you'll need to modify patterns used to match `files`.

 **BEFORE**

@@ -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@v22.2
+        uses: tj-actions/changed-files@v23.1
        with:
          files: |
            *.sh
@@ -439,6 +439,8 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
    <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>
    <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
@@ -9,16 +9,23 @@ if [[ -n $INPUT_PATH ]]; then

  echo "::debug::Resolving repository path: $REPO_DIR"
  if [[ ! -d "$REPO_DIR" ]]; then
-    echo "::warning::Invalid repository path: $REPO_DIR"
+    echo "::error::Invalid repository path: $REPO_DIR"
    exit 1
  fi
  cd "$REPO_DIR"
 fi

+git --version 1>/dev/null 2>&1 && exit_status=$? || exit_status=$?
+
+if [[ $exit_status -ne 0 ]]; then
+  echo "::error::git not installed"
+  exit 1
+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
@@ -26,66 +33,86 @@ fi
 git rev-parse --quiet --verify "$CURRENT_SHA^{commit}" 1>/dev/null 2>&1 && exit_status=$? || exit_status=$?

 if [[ $exit_status -ne 0 ]]; then
-  echo "::warning::Unable to locate the current sha: $CURRENT_SHA"
-  echo "::warning::You seem to be missing 'fetch-depth: 0' or 'fetch-depth: 2'. See https://github.com/tj-actions/changed-files#usage"
+  echo "::error::Unable to locate the current sha: $CURRENT_SHA"
+  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
+    git fetch --no-tags -u --progress --depth=2 origin "${CURRENT_BRANCH}":"${CURRENT_BRANCH}" && exit_status=$? || exit_status=$?
+
+    if [[ $(git rev-list --count "HEAD") -gt 1 ]]; then
+      PREVIOUS_SHA=$(git rev-list -n 1 "HEAD^1" 2>&1) && exit_status=$? || exit_status=$?
+      echo "::debug::Previous SHA: $PREVIOUS_SHA"
    else
      PREVIOUS_SHA=$CURRENT_SHA && exit_status=$? || exit_status=$?
      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=$?

  if [[ $exit_status -ne 0 ]]; then
-    echo "::warning::Unable to locate the previous sha: $PREVIOUS_SHA"
-    echo "::warning::You seem to be missing 'fetch-depth: 0' or 'fetch-depth: 2'. See https://github.com/tj-actions/changed-files#usage"
+    echo "::error::Unable to locate the previous sha: $PREVIOUS_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
 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..."
  git rev-parse --quiet --verify "$PREVIOUS_SHA^{commit}" 1>/dev/null 2>&1 && exit_status=$? || exit_status=$?

  if [[ $exit_status -ne 0 ]]; then
-    echo "::warning::Unable to locate the previous sha: $PREVIOUS_SHA"
-    echo "::warning::You seem to be missing 'fetch-depth: 0' or 'fetch-depth: 2'. See https://github.com/tj-actions/changed-files#usage"
+    echo "::error::Unable to locate the previous sha: $PREVIOUS_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
 fi

+if [[ -n "$PREVIOUS_SHA" && -n "$CURRENT_SHA" && "$PREVIOUS_SHA" == "$CURRENT_SHA" ]]; 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"
--- a/entrypoint.sh
+++ b/entrypoint.sh
@@ -71,7 +71,7 @@ if [[ -n $INPUT_PATH ]]; then

  echo "::debug::Resolving repository path: $REPO_DIR"
  if [[ ! -d "$REPO_DIR" ]]; then
-    echo "::warning::Invalid repository path: $REPO_DIR"
+    echo "::error::Invalid repository path: $REPO_DIR"
    exit 1
  fi
  cd "$REPO_DIR"
Author	SHA1	Message	Date
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
Tonye Jack	34a865a2b2	chore: removed unused code (#534 )	2022-06-24 00:19:45 -04:00
Tonye Jack	3df9c18651	chore: improve error handling (#533 ) * chore: improve error handling * Update test.yml * Update test.yml * Update test.yml * Update test.yml * Update test.yml * Update diff-sha.sh * Update diff-sha.sh * updated command. * Updated message level. * Updated message. * Updated message.	2022-06-24 00:17:41 -04:00
renovate[bot]	3fac642ac6	chore(deps): update tj-actions/verify-changed-files action to v10	2022-06-22 07:56:35 -04:00
Tonye Jack	38cf4def05	Updated README.md (#528 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-06-15 10:12:05 -04:00
Renovate Bot	0d740caef8	chore(deps): update codacy/codacy-analysis-cli-action action to v4.1.0	2022-06-15 09:41:32 -04:00
allcontributors[bot]	d9101bf951	docs: add deronnax as a contributor for doc (#526 ) * docs: update README.md [skip ci] * docs: update .all-contributorsrc [skip ci] Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Co-authored-by: Tonye Jack <jtonye@ymail.com>	2022-06-13 08:58:37 -04:00
Mathieu Dupuy	4d9c041938	fix mispellings (#525 )	2022-06-13 08:57:36 -04:00
Tonye Jack	ad2f98ecee	chore: reformat manual-test.yml (#524 )	2022-06-13 00:09:49 -04:00
Tonye Jack	d95d0359a5	Update README.md	2022-06-13 00:00:23 -04:00
Tonye Jack	4221d49190	Update README.md	2022-06-12 23:58:54 -04:00
Tonye Jack	6a63d1e1bf	Upgraded from v22.2 -> v23 (#523 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2022-06-12 13:33:56 -04:00