chore(deps): bump test/demo from e168fac to f0065d7 (#1481 )

Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
chore(deps-dev): bump @types/node from 20.5.3 to 20.5.4 (#1480 )
2023-08-23 22:40:11 +00:00 · 2023-08-23 21:46:49 +00:00 · 2023-08-23 15:09:57 -06:00 · 2023-08-23 15:05:35 -06:00 · 2023-08-23 15:04:44 -06:00 · 2023-08-23 05:18:13 +00:00
14 changed files with 328 additions and 242 deletions
--- a/.github/dependabot.yml
+++ b/.github/dependabot.yml
@@ -15,3 +15,10 @@ updates:
  open-pull-requests-limit: 10
  labels:
    - "merge when passing"
+- package-ecosystem: gitsubmodule
+  directory: /
+  schedule:
+    interval: daily
+  open-pull-requests-limit: 10
+  labels:
+    - "merge when passing"
--- a/.github/workflows/submodule-sync.yml
+++ b/.github/workflows/submodule-sync.yml
@@ -1,29 +0,0 @@
-on:
-  workflow_dispatch:
-
-jobs:
-  sync:
-    name: Submodule Sync
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-          submodules: recursive
-
-      - name: Git Sumbodule Update
-        run: |
-          git pull --recurse-submodules
-          git submodule update --remote --recursive
-      
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5.0.2
-        with:
-          title: "Updated submodule"
-          labels: "merge when passing"
-          branch: "chore/update-submodule"
-          commit-message: "Updated submodule"
-          body: "Updated submodule"
-          token: ${{ secrets.PAT_TOKEN }}
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -82,7 +82,6 @@ jobs:
      - name: Verify Changed files
        uses: tj-actions/verify-changed-files@v16
        id: changed_files
-        if: github.event_name == 'pull_request'
        with:
          files: |
            src
--- a/HISTORY.md
+++ b/HISTORY.md
@@ -1,5 +1,42 @@
 # Changelog

+# [38.0.0](https://github.com/tj-actions/changed-files/compare/v37.6.1...v38.0.0) - (2023-08-23)
+
+## <!-- 0 -->🚀 Features
+
+- Add support for including matching changed files when dir_names is set to true ([#1464](https://github.com/tj-actions/changed-files/issues/1464)) ([8789204](https://github.com/tj-actions/changed-files/commit/8789204f9747de46e322edc26ccef589722b5d12))  - (Tonye Jack)
+
+## <!-- 1 -->🐛 Bug Fixes
+
+- Error getting diff for submodules ([#1473](https://github.com/tj-actions/changed-files/issues/1473)) ([17f3fec](https://github.com/tj-actions/changed-files/commit/17f3fec1edef0c3916d59cbcee1585fcd457e456))  - (Tonye Jack)
+- Bug with locating the previous tag ([#1470](https://github.com/tj-actions/changed-files/issues/1470)) ([fc63326](https://github.com/tj-actions/changed-files/commit/fc633265449c7430ee5634b1c673247b313ef8cf))  - (Tonye Jack)
+
+## <!-- 26 -->🔄 Update
+
+- Updated README.md ([#1466](https://github.com/tj-actions/changed-files/issues/1466))
+
+Co-authored-by: repo-ranger[bot] <repo-ranger[bot]@users.noreply.github.com> ([e8f1278](https://github.com/tj-actions/changed-files/commit/e8f12786b4779b5208c8e0593863215f52c804e0))  - (tj-actions[bot])
+- Update README.md ([f8b4039](https://github.com/tj-actions/changed-files/commit/f8b4039d9628a18c5e26b11a098230dd037e299e))  - (Tonye Jack)
+- Update README.md ([0e8eddf](https://github.com/tj-actions/changed-files/commit/0e8eddfd8c525e4d8522d7de71af8ad5e4290b90))  - (Tonye Jack)
+
+## <!-- 7 -->⚙️ Miscellaneous Tasks
+
+- Update test ([#1469](https://github.com/tj-actions/changed-files/issues/1469)) ([52dbf3e](https://github.com/tj-actions/changed-files/commit/52dbf3e3cf799a1324a9c148c64948abbf8c3fa6))  - (Tonye Jack)
+- **deps:** Update dependency @types/jest to v29.5.4 ([#1472](https://github.com/tj-actions/changed-files/issues/1472)) ([0a814e4](https://github.com/tj-actions/changed-files/commit/0a814e4372753d6beb4a9ec8f4166aea312f679a))  - (renovate[bot])
+- **deps:** Update dependency @types/node to v20.5.3 ([#1467](https://github.com/tj-actions/changed-files/issues/1467)) ([18bacaa](https://github.com/tj-actions/changed-files/commit/18bacaab15227e761b04fb4097648318590be71a))  - (renovate[bot])
+- Update .eslintrc.json ([#1465](https://github.com/tj-actions/changed-files/issues/1465)) ([819fb64](https://github.com/tj-actions/changed-files/commit/819fb645201f8693c48ca1158b4069d0c7cec64d))  - (Tonye Jack)
+- **deps:** Update typescript-eslint monorepo to v6.4.1 ([#1463](https://github.com/tj-actions/changed-files/issues/1463)) ([31003b4](https://github.com/tj-actions/changed-files/commit/31003b45e6faa01151aeb2508de21d895b9024d8))  - (renovate[bot])
+- **deps:** Update dependency jest to v29.6.3 ([#1462](https://github.com/tj-actions/changed-files/issues/1462)) ([1f04b16](https://github.com/tj-actions/changed-files/commit/1f04b16682ff06e18be316b672337f43ebd8f768))  - (renovate[bot])
+- **deps:** Lock file maintenance ([#1461](https://github.com/tj-actions/changed-files/issues/1461)) ([0a8b04e](https://github.com/tj-actions/changed-files/commit/0a8b04e3e345eae4a8aa28fb46959c1509197ee5))  - (renovate[bot])
+- **deps:** Update dependency @types/node to v20.5.1 ([#1460](https://github.com/tj-actions/changed-files/issues/1460)) ([6da496e](https://github.com/tj-actions/changed-files/commit/6da496e2f92930ffc11735db0b53e4e2027e9e26))  - (renovate[bot])
+- **deps:** Update actions/setup-node action to v3.8.1 ([#1459](https://github.com/tj-actions/changed-files/issues/1459)) ([68d38a3](https://github.com/tj-actions/changed-files/commit/68d38a3f163ab827c733c45c9f23d607401a34ce))  - (renovate[bot])
+
+## <!-- 9 -->⬆️ Upgrades
+
+- Upgraded to v37.6.1 ([#1458](https://github.com/tj-actions/changed-files/issues/1458))
+
+Co-authored-by: jackton1 <jackton1@users.noreply.github.com> ([3645cf9](https://github.com/tj-actions/changed-files/commit/3645cf9fc374b6e501b12888142f2aabc6b3cbf9))  - (tj-actions[bot])
+
 # [37.6.1](https://github.com/tj-actions/changed-files/compare/v37.6.0...v37.6.1) - (2023-08-15)

 ## <!-- 26 -->🔄 Update
--- a/README.md
+++ b/README.md
@@ -111,7 +111,7 @@ jobs:
      # Example 1
      - name: Get all test, doc and src files that have changed
        id: changed-files-yaml
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38
        with:
          files_yaml: |
            doc:
@@ -141,7 +141,7 @@ jobs:
      # Example 2
      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38
        
        # To compare changes between the current commit and the last pushed remote commit set `since_last_remote_commit: true`. e.g
        # with:
@@ -156,7 +156,7 @@ jobs:
      # Example 3
      - name: Get changed files in the docs folder
        id: changed-files-specific
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38
        with:
          files: docs/*.{js,html}  # Alternatively using: `docs/**` or `docs`
          files_ignore: docs/static.js
@@ -196,7 +196,7 @@ jobs:
    steps:
      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38

      - name: List all changed files
        run: |
@@ -256,7 +256,7 @@ jobs:
      # Example 1
      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38

      # NOTE: `since_last_remote_commit: true` is implied by default and falls back to the previous local commit.

@@ -362,6 +362,7 @@ Support this project with a :star:
 |      dir\_names\_include\_files\_separator       | string |  false   |          `"\n"`           |                                                                                                       Separator used to split the <br>`dir_names_include_files` input                                                                                                        |
 |             dir\_names\_max\_depth              | string |  false   |                           |                                                                        Limit the directory output to <br>a maximum depth e.g `test/test1/test2` <br>with max depth of `2` <br>returns `test/test1`.                                                                          |
 |                 escape\_json                  | string |  false   |         `"true"`          |                                                                                                                             Escape JSON output.                                                                                                                              |
+|      fetch\_additional\_submodule\_history      | string |  false   |         `"false"`         |                                                                                                                   Fetch additional history for submodules.                                                                                                                   |
 |                 fetch\_depth                  | string |  false   |          `"50"`           |                                                                     Depth of additional branch history <br>fetched. **NOTE**: This can be <br>adjusted to resolve errors with <br>insufficient history.                                                                      |
 |                    files                     | string |  false   |                           |                                                File and directory patterns used <br>to detect changes (Defaults to the entire repo if unset) **NOTE:** <br>Multiline file/directory patterns should not <br>include quotes.                                                  |
 |            files\_from\_source\_file            | string |  false   |                           |                                                                                                           Source file(s) used to populate <br>the `files` input.                                                                                                             |
@@ -423,7 +424,7 @@ The format of the version string is as follows:
 ...
    - name: Get changed files
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
 ...
 ```

@@ -436,7 +437,7 @@ The format of the version string is as follows:
 ...
    - name: Get all changed files and use a comma separator in the output
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        separator: ","
 ...
@@ -453,7 +454,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed files
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38

    - name: List all added files
      run: |
@@ -474,7 +475,7 @@ See [outputs](#outputs) for a list of all available outputs.
 ...
    - name: Get changed files
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38

    - name: Run a step if my-file.txt was modified
      if: contains(steps.changed-files.outputs.modified_files, 'my-file.txt')
@@ -534,7 +535,7 @@ See [outputs](#outputs) for a list of all available outputs.
 ...
    - name: Get changed files
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        files: |
          my-file.txt
@@ -557,7 +558,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed files
      id: changed-files-specific
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        files: |
          my-file.txt
@@ -604,7 +605,7 @@ See [outputs](#outputs) for a list of all available outputs.
 ...
    - name: Get changed files using a source file or list of file(s) to populate to files input.
      id: changed-files-specific-source-file
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        files_from_source_file: test/changed-files-list.txt
 ...
@@ -621,7 +622,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed files using 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@v37
+      uses: tj-actions/changed-files@v38
      with:
        files_from_source_file: |
          test/changed-files-list.txt
@@ -642,7 +643,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed files using a different SHA
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        sha: ${{ github.event.pull_request.head.sha }}
 ...
@@ -659,7 +660,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed files using a different base SHA
      id: changed-files
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        base_sha: ${{ github.event.pull_request.base.sha }}
 ...
@@ -691,11 +692,11 @@ jobs:

      - name: Get changed files
        id: changed-files
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38

      - name: Get changed files in the .github folder
        id: changed-files-specific
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38
        with:
          base_sha: ${{ steps.get-base-sha.outputs.base_sha }}
          files: .github/**
@@ -725,7 +726,7 @@ See [inputs](#inputs) for more information.

    - name: Run changed-files with defaults in dir1
      id: changed-files-for-dir1
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        path: dir1

@@ -748,7 +749,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Run changed-files with quotepath disabled
      id: changed-files-quotepath
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        quotepath: "false"

@@ -787,7 +788,7 @@ See [inputs](#inputs) for more information.

      - 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@v37
+        uses: tj-actions/changed-files@v38
        with:
          base_sha: ${{ steps.last_successful_commit_push.outputs.base }}
 ...
@@ -814,7 +815,7 @@ See [inputs](#inputs) for more information.

      - name: Run changed-files with the commit of the last successful test workflow run on the main branch
        id: changed-files-base-sha-pull-request
-        uses: tj-actions/changed-files@v37
+        uses: tj-actions/changed-files@v38
        with:
          base_sha: ${{ steps.last_successful_commit_pull_request.outputs.base }}
 ...
@@ -840,7 +841,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Run changed-files with dir_names
      id: changed-files-dir-names
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        dir_names: "true"
 ...
@@ -857,7 +858,7 @@ See [inputs](#inputs) for more information.
 ...
    - name: Run changed-files with JSON output
      id: changed-files-json
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        json: "true"
 ...
@@ -874,13 +875,13 @@ See [inputs](#inputs) for more information.
 ...
    - name: Get changed-files since 2022-08-19
      id: changed-files-since
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        since: "2022-08-19"

    - name: Get changed-files until 2022-08-20
      id: changed-files-until
-      uses: tj-actions/changed-files@v37
+      uses: tj-actions/changed-files@v38
      with:
        until: "2022-08-20"
 ...
--- a/action.yml
+++ b/action.yml
@@ -136,6 +136,10 @@ inputs:
    description: "Skip the initial fetch to improve performance for shallow repositories. **NOTE**: This could lead to errors with missing history and the intended use is limited to when you've fetched the history necessary to perform the diff."
    required: false
    default: "false"
+  fetch_additional_submodule_history:
+    description: "Fetch additional history for submodules."
+    required: false
+    default: "false"
  since_last_remote_commit:
    description: "Use the last commit on the remote branch as the `base_sha`. Defaults to the last non-merge commit on the target branch for pull request events and the previous remote commit of the current branch for push events."
    required: false
--- a/dist/index.js
+++ b/dist/index.js
@@ -82,6 +82,7 @@ const getRenamedFiles = ({ inputs, workingDirectory, hasSubmodule, diffResult, s
                    sha2: submoduleShaResult.currentSha,
                    diff
                }))) {
+                    core.warning(`Set 'fetch_additional_submodule_history: true' to fetch additional submodule history for: ${submodulePath}, Note you can control the fetch depth using 'fetch_depth' input`);
                    diff = '..';
                }
                const submoduleRenamedFiles = yield (0, utils_1.gitRenamedFiles)({
@@ -146,6 +147,7 @@ const getAllDiffFiles = ({ workingDirectory, hasSubmodule, diffResult, submodule
                    sha2: submoduleShaResult.currentSha,
                    diff
                }))) {
+                    core.warning(`Set 'fetch_additional_submodule_history: true' to fetch additional submodule history for: ${submodulePath}, Note you can control the fetch depth using 'fetch_depth' input`);
                    diff = '..';
                }
                const submoduleFiles = yield (0, utils_1.getAllChangedFiles)({
@@ -753,51 +755,66 @@ const getSHAForNonPullRequestEvent = (inputs, env, workingDirectory, isShallow,
    let targetBranch = env.GITHUB_REF_NAME;
    const currentBranch = targetBranch;
    let initialCommit = false;
-    if (isShallow && !inputs.skipInitialFetch) {
-        core.info('Repository is shallow, fetching more history...');
-        if (isTag) {
-            let sourceBranch = '';
-            if (github.context.payload.base_ref) {
-                sourceBranch = github.context.payload.base_ref.replace('refs/heads/', '');
+    if (!inputs.skipInitialFetch) {
+        if (isShallow) {
+            core.info('Repository is shallow, fetching more history...');
+            if (isTag) {
+                let sourceBranch = '';
+                if (github.context.payload.base_ref) {
+                    sourceBranch = github.context.payload.base_ref.replace('refs/heads/', '');
+                }
+                else if ((_h = github.context.payload.release) === null || _h === void 0 ? void 0 : _h.target_commitish) {
+                    sourceBranch = (_j = github.context.payload.release) === null || _j === void 0 ? void 0 : _j.target_commitish;
+                }
+                yield (0, utils_1.gitFetch)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`,
+                        'origin',
+                        `+refs/heads/${sourceBranch}:refs/remotes/origin/${sourceBranch}`
+                    ]
+                });
            }
-            else if ((_h = github.context.payload.release) === null || _h === void 0 ? void 0 : _h.target_commitish) {
-                sourceBranch = (_j = github.context.payload.release) === null || _j === void 0 ? void 0 : _j.target_commitish;
+            else {
+                yield (0, utils_1.gitFetch)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`,
+                        'origin',
+                        `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
+                    ]
+                });
+            }
+            if (hasSubmodule) {
+                yield (0, utils_1.gitFetchSubmodules)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`
+                    ]
+                });
            }
-            yield (0, utils_1.gitFetch)({
-                cwd: workingDirectory,
-                args: [
-                    ...gitFetchExtraArgs,
-                    '-u',
-                    '--progress',
-                    `--deepen=${inputs.fetchDepth}`,
-                    'origin',
-                    `+refs/heads/${sourceBranch}:refs/remotes/origin/${sourceBranch}`
-                ]
-            });
        }
        else {
-            yield (0, utils_1.gitFetch)({
-                cwd: workingDirectory,
-                args: [
-                    ...gitFetchExtraArgs,
-                    '-u',
-                    '--progress',
-                    `--deepen=${inputs.fetchDepth}`,
-                    'origin',
-                    `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
-                ]
-            });
-        }
-        if (hasSubmodule) {
-            yield (0, utils_1.gitFetchSubmodules)({
-                cwd: workingDirectory,
-                args: [
-                    ...gitFetchExtraArgs,
-                    '-u',
-                    '--progress',
-                    `--deepen=${inputs.fetchDepth}`
-                ]
-            });
+            if (hasSubmodule && inputs.fetchSubmoduleHistory) {
+                yield (0, utils_1.gitFetchSubmodules)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`
+                    ]
+                });
+            }
        }
    }
    const currentSha = yield getCurrentSHA({ inputs, workingDirectory });
@@ -910,48 +927,63 @@ const getSHAForPullRequestEvent = (inputs, env, workingDirectory, isShallow, has
    if (inputs.sinceLastRemoteCommit) {
        targetBranch = currentBranch;
    }
-    if (isShallow && !inputs.skipInitialFetch) {
+    if (!inputs.skipInitialFetch) {
        core.info('Repository is shallow, fetching more history...');
-        let prFetchExitCode = yield (0, utils_1.gitFetch)({
-            cwd: workingDirectory,
-            args: [
-                ...gitFetchExtraArgs,
-                '-u',
-                '--progress',
-                'origin',
-                `pull/${(_q = github.context.payload.pull_request) === null || _q === void 0 ? void 0 : _q.number}/head:${currentBranch}`
-            ]
-        });
-        if (prFetchExitCode !== 0) {
-            prFetchExitCode = yield (0, utils_1.gitFetch)({
+        if (isShallow) {
+            let prFetchExitCode = yield (0, utils_1.gitFetch)({
                cwd: workingDirectory,
                args: [
                    ...gitFetchExtraArgs,
                    '-u',
                    '--progress',
-                    `--deepen=${inputs.fetchDepth}`,
                    'origin',
-                    `+refs/heads/${currentBranch}*:refs/remotes/origin/${currentBranch}*`
+                    `pull/${(_q = github.context.payload.pull_request) === null || _q === void 0 ? void 0 : _q.number}/head:${currentBranch}`
                ]
            });
+            if (prFetchExitCode !== 0) {
+                prFetchExitCode = yield (0, utils_1.gitFetch)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`,
+                        'origin',
+                        `+refs/heads/${currentBranch}*:refs/remotes/origin/${currentBranch}*`
+                    ]
+                });
+            }
+            if (prFetchExitCode !== 0) {
+                throw new Error('Failed to fetch pull request branch. Please ensure "persist-credentials" is set to "true" when checking out the repository. See: https://github.com/actions/checkout#usage');
+            }
+            if (!inputs.sinceLastRemoteCommit) {
+                core.debug('Fetching target branch...');
+                yield (0, utils_1.gitFetch)({
+                    cwd: workingDirectory,
+                    args: [
+                        ...gitFetchExtraArgs,
+                        '-u',
+                        '--progress',
+                        `--deepen=${inputs.fetchDepth}`,
+                        'origin',
+                        `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
+                    ]
+                });
+                if (hasSubmodule) {
+                    yield (0, utils_1.gitFetchSubmodules)({
+                        cwd: workingDirectory,
+                        args: [
+                            ...gitFetchExtraArgs,
+                            '-u',
+                            '--progress',
+                            `--deepen=${inputs.fetchDepth}`
+                        ]
+                    });
+                }
+            }
        }
-        if (prFetchExitCode !== 0) {
-            throw new Error('Failed to fetch pull request branch. Please ensure "persist-credentials" is set to "true" when checking out the repository. See: https://github.com/actions/checkout#usage');
-        }
-        if (!inputs.sinceLastRemoteCommit) {
-            core.debug('Fetching target branch...');
-            yield (0, utils_1.gitFetch)({
-                cwd: workingDirectory,
-                args: [
-                    ...gitFetchExtraArgs,
-                    '-u',
-                    '--progress',
-                    `--deepen=${inputs.fetchDepth}`,
-                    'origin',
-                    `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
-                ]
-            });
-            if (hasSubmodule) {
+        else {
+            if (hasSubmodule && inputs.fetchSubmoduleHistory) {
                yield (0, utils_1.gitFetchSubmodules)({
                    cwd: workingDirectory,
                    args: [
@@ -1288,6 +1320,9 @@ const getInputs = () => {
    const skipInitialFetch = core.getBooleanInput('skip_initial_fetch', {
        required: false
    });
+    const fetchSubmoduleHistory = core.getBooleanInput('fetch_additional_submodule_history', {
+        required: false
+    });
    const inputs = {
        files,
        filesSeparator,
@@ -1323,6 +1358,7 @@ const getInputs = () => {
        oldNewSeparator,
        oldNewFilesSeparator,
        skipInitialFetch,
+        fetchSubmoduleHistory,
        // End Not Supported via REST API
        dirNames,
        dirNamesExcludeCurrentDir,
@@ -1596,7 +1632,8 @@ function run() {
                'recoverFiles',
                'recoverFilesIgnore',
                'includeAllOldNewRenamedFiles',
-                'skipInitialFetch'
+                'skipInitialFetch',
+                'fetchSubmoduleHistory'
            ];
            for (const input of unsupportedInputs) {
                if (inputs[input]) {
--- a/dist/index.js.map
+++ b/dist/index.js.map
--- a/src/changedFiles.ts
+++ b/src/changedFiles.ts
@@ -65,6 +65,9 @@ export const getRenamedFiles = async ({
            diff
          }))
        ) {
+          core.warning(
+            `Set 'fetch_additional_submodule_history: true' to fetch additional submodule history for: ${submodulePath}, Note you can control the fetch depth using 'fetch_depth' input`
+          )
          diff = '..'
        }

@@ -157,6 +160,9 @@ export const getAllDiffFiles = async ({
            diff
          }))
        ) {
+          core.warning(
+            `Set 'fetch_additional_submodule_history: true' to fetch additional submodule history for: ${submodulePath}, Note you can control the fetch depth using 'fetch_depth' input`
+          )
          diff = '..'
        }

--- a/src/commitSha.ts
+++ b/src/commitSha.ts
@@ -93,56 +93,70 @@ export const getSHAForNonPullRequestEvent = async (
  const currentBranch = targetBranch
  let initialCommit = false

-  if (isShallow && !inputs.skipInitialFetch) {
-    core.info('Repository is shallow, fetching more history...')
+  if (!inputs.skipInitialFetch) {
+    if (isShallow) {
+      core.info('Repository is shallow, fetching more history...')

-    if (isTag) {
-      let sourceBranch = ''
+      if (isTag) {
+        let sourceBranch = ''

-      if (github.context.payload.base_ref) {
-        sourceBranch = github.context.payload.base_ref.replace(
-          'refs/heads/',
-          ''
-        )
-      } else if (github.context.payload.release?.target_commitish) {
-        sourceBranch = github.context.payload.release?.target_commitish
+        if (github.context.payload.base_ref) {
+          sourceBranch = github.context.payload.base_ref.replace(
+            'refs/heads/',
+            ''
+          )
+        } else if (github.context.payload.release?.target_commitish) {
+          sourceBranch = github.context.payload.release?.target_commitish
+        }
+
+        await gitFetch({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`,
+            'origin',
+            `+refs/heads/${sourceBranch}:refs/remotes/origin/${sourceBranch}`
+          ]
+        })
+      } else {
+        await gitFetch({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`,
+            'origin',
+            `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
+          ]
+        })
      }

-      await gitFetch({
-        cwd: workingDirectory,
-        args: [
-          ...gitFetchExtraArgs,
-          '-u',
-          '--progress',
-          `--deepen=${inputs.fetchDepth}`,
-          'origin',
-          `+refs/heads/${sourceBranch}:refs/remotes/origin/${sourceBranch}`
-        ]
-      })
+      if (hasSubmodule) {
+        await gitFetchSubmodules({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`
+          ]
+        })
+      }
    } else {
-      await gitFetch({
-        cwd: workingDirectory,
-        args: [
-          ...gitFetchExtraArgs,
-          '-u',
-          '--progress',
-          `--deepen=${inputs.fetchDepth}`,
-          'origin',
-          `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
-        ]
-      })
-    }
-
-    if (hasSubmodule) {
-      await gitFetchSubmodules({
-        cwd: workingDirectory,
-        args: [
-          ...gitFetchExtraArgs,
-          '-u',
-          '--progress',
-          `--deepen=${inputs.fetchDepth}`
-        ]
-      })
+      if (hasSubmodule && inputs.fetchSubmoduleHistory) {
+        await gitFetchSubmodules({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`
+          ]
+        })
+      }
    }
  }

@@ -288,55 +302,68 @@ export const getSHAForPullRequestEvent = async (
    targetBranch = currentBranch
  }

-  if (isShallow && !inputs.skipInitialFetch) {
+  if (!inputs.skipInitialFetch) {
    core.info('Repository is shallow, fetching more history...')
-
-    let prFetchExitCode = await gitFetch({
-      cwd: workingDirectory,
-      args: [
-        ...gitFetchExtraArgs,
-        '-u',
-        '--progress',
-        'origin',
-        `pull/${github.context.payload.pull_request?.number}/head:${currentBranch}`
-      ]
-    })
-
-    if (prFetchExitCode !== 0) {
-      prFetchExitCode = await gitFetch({
+    if (isShallow) {
+      let prFetchExitCode = await gitFetch({
        cwd: workingDirectory,
        args: [
          ...gitFetchExtraArgs,
          '-u',
          '--progress',
-          `--deepen=${inputs.fetchDepth}`,
          'origin',
-          `+refs/heads/${currentBranch}*:refs/remotes/origin/${currentBranch}*`
-        ]
-      })
-    }
-
-    if (prFetchExitCode !== 0) {
-      throw new Error(
-        'Failed to fetch pull request branch. Please ensure "persist-credentials" is set to "true" when checking out the repository. See: https://github.com/actions/checkout#usage'
-      )
-    }
-
-    if (!inputs.sinceLastRemoteCommit) {
-      core.debug('Fetching target branch...')
-      await gitFetch({
-        cwd: workingDirectory,
-        args: [
-          ...gitFetchExtraArgs,
-          '-u',
-          '--progress',
-          `--deepen=${inputs.fetchDepth}`,
-          'origin',
-          `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
+          `pull/${github.context.payload.pull_request?.number}/head:${currentBranch}`
        ]
      })

-      if (hasSubmodule) {
+      if (prFetchExitCode !== 0) {
+        prFetchExitCode = await gitFetch({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`,
+            'origin',
+            `+refs/heads/${currentBranch}*:refs/remotes/origin/${currentBranch}*`
+          ]
+        })
+      }
+
+      if (prFetchExitCode !== 0) {
+        throw new Error(
+          'Failed to fetch pull request branch. Please ensure "persist-credentials" is set to "true" when checking out the repository. See: https://github.com/actions/checkout#usage'
+        )
+      }
+
+      if (!inputs.sinceLastRemoteCommit) {
+        core.debug('Fetching target branch...')
+        await gitFetch({
+          cwd: workingDirectory,
+          args: [
+            ...gitFetchExtraArgs,
+            '-u',
+            '--progress',
+            `--deepen=${inputs.fetchDepth}`,
+            'origin',
+            `+refs/heads/${targetBranch}:refs/remotes/origin/${targetBranch}`
+          ]
+        })
+
+        if (hasSubmodule) {
+          await gitFetchSubmodules({
+            cwd: workingDirectory,
+            args: [
+              ...gitFetchExtraArgs,
+              '-u',
+              '--progress',
+              `--deepen=${inputs.fetchDepth}`
+            ]
+          })
+        }
+      }
+    } else {
+      if (hasSubmodule && inputs.fetchSubmoduleHistory) {
        await gitFetchSubmodules({
          cwd: workingDirectory,
          args: [
--- a/src/inputs.ts
+++ b/src/inputs.ts
@@ -34,6 +34,7 @@ export type Inputs = {
  json: boolean
  escapeJson: boolean
  fetchDepth?: number
+  fetchSubmoduleHistory?: boolean
  sinceLastRemoteCommit: boolean
  writeOutputFiles: boolean
  outputDir: string
@@ -189,6 +190,12 @@ export const getInputs = (): Inputs => {
  const skipInitialFetch = core.getBooleanInput('skip_initial_fetch', {
    required: false
  })
+  const fetchSubmoduleHistory = core.getBooleanInput(
+    'fetch_additional_submodule_history',
+    {
+      required: false
+    }
+  )

  const inputs: Inputs = {
    files,
@@ -225,6 +232,7 @@ export const getInputs = (): Inputs => {
    oldNewSeparator,
    oldNewFilesSeparator,
    skipInitialFetch,
+    fetchSubmoduleHistory,
    // End Not Supported via REST API
    dirNames,
    dirNamesExcludeCurrentDir,
--- a/src/main.ts
+++ b/src/main.ts
@@ -317,7 +317,8 @@ export async function run(): Promise<void> {
      'recoverFiles',
      'recoverFilesIgnore',
      'includeAllOldNewRenamedFiles',
-      'skipInitialFetch'
+      'skipInitialFetch',
+      'fetchSubmoduleHistory'
    ]

    for (const input of unsupportedInputs) {
--- a/test/demo
+++ b/test/demo
--- a/yarn.lock
+++ b/yarn.lock
@@ -354,9 +354,9 @@
    eslint-visitor-keys "^3.3.0"

 "@eslint-community/regexpp@^4.5.1", "@eslint-community/regexpp@^4.6.1":
-  version "4.6.2"
-  resolved "https://registry.yarnpkg.com/@eslint-community/regexpp/-/regexpp-4.6.2.tgz#1816b5f6948029c5eaacb0703b850ee0cb37d8f8"
-  integrity sha512-pPTNuaAG3QMH+buKyBIGJs3g/S5y0caxw0ygM3YyE6yJFySwiGGSzA+mM3KJ8QQvzeLh3blwgSonkFjgQdxzMw==
+  version "4.7.0"
+  resolved "https://registry.yarnpkg.com/@eslint-community/regexpp/-/regexpp-4.7.0.tgz#96e7c05e738327602ae5942437f9c6b177ec279a"
+  integrity sha512-+HencqxU7CFJnQb7IKtuNBqS6Yx3Tz4kOL8BJXo+JyeiBm5MEX6pO8onXDkjrkCRlfYXS1Axro15ZjVFe9YgsA==

 "@eslint/eslintrc@^2.1.2":
  version "2.1.2"
@@ -598,7 +598,7 @@
    slash "^3.0.0"
    write-file-atomic "^4.0.2"

-"@jest/types@^29.6.1", "@jest/types@^29.6.3":
+"@jest/types@^29.6.3":
  version "29.6.3"
  resolved "https://registry.yarnpkg.com/@jest/types/-/types-29.6.3.tgz#1131f8cf634e7e84c5e77bab12f052af585fba59"
  integrity sha512-u3UPsIilWKOM3F9CXtrG8LEJmNxwoCQC/XVj4IKYXvvpx7QIi/Kg1LI5uDmDpKlac62NUtX7eLjRh+jVZcLOzw==
@@ -986,9 +986,9 @@
    "@types/braces" "*"

 "@types/node@*", "@types/node@^20.3.2":
-  version "20.5.3"
-  resolved "https://registry.yarnpkg.com/@types/node/-/node-20.5.3.tgz#fa52c147f405d56b2f1dd8780d840aa87ddff629"
-  integrity sha512-ITI7rbWczR8a/S6qjAW7DMqxqFMjjTo61qZVWJ1ubPvbIQsL5D/TvwjYEalM8Kthpe3hTzOGrF2TGbAu2uyqeA==
+  version "20.5.4"
+  resolved "https://registry.yarnpkg.com/@types/node/-/node-20.5.4.tgz#4666fb40f9974d60c53c4ff554315860ba4feab8"
+  integrity sha512-Y9vbIAoM31djQZrPYjpTLo0XlaSwOIsrlfE3LpulZeRblttsLQRFRlBAppW0LOxyT3ALj2M5vU1ucQQayQH3jA==

 "@types/semver@^7.3.12", "@types/semver@^7.5.0":
  version "7.5.0"
@@ -1689,9 +1689,9 @@ doctrine@^3.0.0:
    esutils "^2.0.2"

 electron-to-chromium@^1.4.477:
-  version "1.4.496"
-  resolved "https://registry.yarnpkg.com/electron-to-chromium/-/electron-to-chromium-1.4.496.tgz#a57534b70d2bdee7e1ad7dbd4c91e560cbd08db1"
-  integrity sha512-qeXC3Zbykq44RCrBa4kr8v/dWzYJA8rAwpyh9Qd+NKWoJfjG5vvJqy9XOJ9H4P/lqulZBCgUWAYi+FeK5AuJ8g==
+  version "1.4.499"
+  resolved "https://registry.yarnpkg.com/electron-to-chromium/-/electron-to-chromium-1.4.499.tgz#dc36b67f4c8e273524e8d2080c5203a6a76987b6"
+  integrity sha512-0NmjlYBLKVHva4GABWAaHuPJolnDuL0AhV3h1hES6rcLCWEIbRL6/8TghfsVwkx6TEroQVdliX7+aLysUpKvjw==

 emittery@^0.13.1:
  version "0.13.1"
@@ -2203,9 +2203,9 @@ fs.realpath@^1.0.0:
  integrity sha512-OO0pH2lK6a0hZnAdau5ItzHPI6pUlvI7jMVnxUQRtw4owF2wk8lOSabtGDCTP4Ggrg2MbGnWO9X8K1t4+fGMDw==

 fsevents@^2.3.2:
-  version "2.3.2"
-  resolved "https://registry.yarnpkg.com/fsevents/-/fsevents-2.3.2.tgz#8a526f78b8fdf4623b709e0b975c52c24c02fd1a"
-  integrity sha512-xiqMQR4xAeHTuB9uWm+fFRcIOgKBMiOBP+eXiyT7jsgVCq1bkVygt00oASowB7EdtpOHaaPgKt812P9ab+DDKA==
+  version "2.3.3"
+  resolved "https://registry.yarnpkg.com/fsevents/-/fsevents-2.3.3.tgz#cac6407785d03675a2a5e1a5305c697b347d90d6"
+  integrity sha512-5xoDfX+fL7faATnagmWPpbFtwh/R77WmMMqqHGS65C3vvB0YHrgF+B1YmZ3441tMj5n63k0212XNoJwzlhffQw==

 function-bind@^1.1.1:
  version "1.1.1"
@@ -2986,19 +2986,7 @@ jest-snapshot@^29.6.3:
    pretty-format "^29.6.3"
    semver "^7.5.3"

-jest-util@^29.0.0:
-  version "29.6.2"
-  resolved "https://registry.yarnpkg.com/jest-util/-/jest-util-29.6.2.tgz#8a052df8fff2eebe446769fd88814521a517664d"
-  integrity sha512-3eX1qb6L88lJNCFlEADKOkjpXJQyZRiavX1INZ4tRnrBVr2COd3RgcTLyUiEXMNBlDU/cgYq6taUS0fExrWW4w==
-  dependencies:
-    "@jest/types" "^29.6.1"
-    "@types/node" "*"
-    chalk "^4.0.0"
-    ci-info "^3.2.0"
-    graceful-fs "^4.2.9"
-    picomatch "^2.2.3"
-
-jest-util@^29.6.3:
+jest-util@^29.0.0, jest-util@^29.6.3:
  version "29.6.3"
  resolved "https://registry.yarnpkg.com/jest-util/-/jest-util-29.6.3.tgz#e15c3eac8716440d1ed076f09bc63ace1aebca63"
  integrity sha512-QUjna/xSy4B32fzcKTSz1w7YYzgiHrjjJjevdRf61HYk998R5vVMMNmrHESYZVDS5DSWs+1srPLPKxXPkeSDOA==
Author	SHA1	Message	Date
dependabot[bot]	1c26215f3f	chore(deps): bump test/demo from `e168fac` to `f0065d7` (#1481 ) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2023-08-23 22:40:11 +00:00
dependabot[bot]	7b1e1540e2	chore(deps-dev): bump @types/node from 20.5.3 to 20.5.4 (#1480 ) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2023-08-23 21:46:49 +00:00
Tonye Jack	02010edac3	chore: move submodule update to dependabot and remove unused workflow (#1478 )	2023-08-23 15:09:57 -06:00
tj-actions[bot]	e4e76943d6	Updated README.md (#1477 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2023-08-23 15:05:35 -06:00
Tonye Jack	569361586a	feat: add support for fetching additional history for submodules (#1476 ) Co-authored-by: GitHub Action <action@github.com>	2023-08-23 15:04:44 -06:00
renovate[bot]	785218258b	chore(deps): lock file maintenance	2023-08-23 05:18:13 +00:00
tj-actions[bot]	0b45bc64c2	Upgraded to v38 (#1474 ) Co-authored-by: jackton1 <jackton1@users.noreply.github.com>	2023-08-22 22:59:10 -06:00