The head commit for this pull_request event is not ahead of the base commit #11

nolanbconaway · 2020-08-18T13:15:40Z

I squashed a bunch of commits from my PR into one, and now the action is giving me a message:

##[error]The head commit for this pull_request event is not ahead of the base commit. Please submit an issue on this action's GitHub repo.
All: [".github/workflows/black.yml"]
Added: [".github/workflows/black.yml"]
Modified: []
Removed: []
Renamed: []
Added or modified: [".github/workflows/black.yml"]

The issue is that the base commit as such no longer exists because it was squashed. However, it appears that the changed files list is still correct. Does this error need to be raised? what needs to happen to handle squashed base/head commits?

The text was updated successfully, but these errors were encountered:

Bertg · 2020-08-25T14:51:25Z

I see the same issue. This is possibly related to #4 and #7

justyn · 2020-09-07T17:19:34Z

I'm also seeing this issue, a lot of failing workflows - could an option be added to drop the error to a warning?

mmalek-sa · 2020-10-28T22:37:27Z

same issue here.

ahoereth · 2020-10-28T22:55:18Z

Workaround:

       - uses: jitterbit/get-changed-files@v1
         id: files
         continue-on-error: true

johentsch · 2020-11-28T17:36:22Z

Same here

dlam · 2020-12-12T03:09:21Z

A reliable way to reproduce this seems to be force push event

javipolo · 2020-12-28T18:02:32Z

Same here ...

Workaround proposed here: jitterbit/get-changed-files#11 (comment)

* Integrate workaround for "head commit not ahead of base commit" Workaround proposed here: jitterbit/get-changed-files#11 (comment) * Vale GHA: filter out files that aren't inside docs/ directory - This helps avoid Vale running on non-content markdown files, like our typography test page. - A new select_docs_dir_files step is added. This step parses and filters the JSON added_modified output from the get_changed_files step. It then sets a new output for this filtered file list that the Vale step later reads from. - The jq command is used to filter the JSON list of files: https://stedolan.github.io/jq/ - This Stack Overflow served as inspiration: https://stackoverflow.com/questions/64482190/edit-the-value-inside-the-json-array-with-github-actions * Only run the Vale step if list of files in docs/ is not empty When the list of files was empty after the select_docs_dir_files step ran, the action would hang at the Vale step: https://github.com/linode/docs/pull/4304/checks?check_run_id=2546802169 * Vale GHA: add comments

Ana06 · 2021-06-04T12:10:33Z

I have cherry-picked 54f3703 (which fixes this issue) and released it in Ana06/get-changed-files@v1.2. I hope this change makes it upstream. In case it doesn't (or until it does), feel free to use Ana06/get-changed-files@v1.2 (which also supports pull_request_target and allow to filter files using regular expressions).

…nged-files#11

When a set of commits is squashed and force pushed, it causes errors with this action. See jitterbit/get-changed-files#11

vmartinv · 2022-02-12T22:05:25Z

I have cherry-picked 54f3703 (which fixes this issue) and released it in Ana06/get-changed-files@v1.2. I hope this change makes it upstream. In case it doesn't (or until it does), feel free to use Ana06/get-changed-files@v1.2 (which also supports pull_request_target and allow to filter files using regular expressions).

this fork fixed the issue for me

sgibson91 · 2022-04-13T09:00:01Z

Would love to see the fork make it into this action!

Bertg · 2022-04-15T07:43:31Z

This issue has been open for a while.

We built an action similar to this one (not entirely the same) here https://github.com/younited/detect-branch-changes-action Maybe this can cover your case as well?

Change to follow jitterbit/get-changed-files#11 and use uses: Ana06/get-changed-files@v1.2

(jitterbit/get-changed-files#11)

see jitterbit/get-changed-files#11 for context when commits are pushed, then later squashed and re-pushed, the action can fail workaround is to continue on error, since the list of files changed is still correct

The previous one seems to have had a bug reported here jitterbit/get-changed-files#11

jitterbit/get-changed-files@v1 often fails with errors like: Base commit: c06f2fe Head commit: 55342d5 Error: The head commit for this pull_request event is not ahead of the base commit. Please submit an issue on this action's GitHub repo. All: components/perf-tools/msr-safe/SPECS/msr-safe.spec Added: Modified: components/perf-tools/msr-safe/SPECS/msr-safe.spec Removed: Renamed: Added or modified: components/perf-tools/msr-safe/SPECS/msr-safe.spec This issue is known since a long time but it seems the action is not maintained anymore: https://github.com/jitterbit/get-changed-files/issues?q=The+head+commit+for+this+pull_request+event+is+not+ahead https://github.com/Ana06/get-changed-files is a maintained fork that claims to have fixed this issue: jitterbit/get-changed-files#11 (comment) Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>

See jitterbit/get-changed-files#11. This action appears to be unmaintained and is superceded by Ana06/get-changed-files.

…uest event is not ahead of the base commit' Re: jitterbit/get-changed-files#11

see jitterbit/get-changed-files#54, jitterbit/get-changed-files#4, jitterbit/get-changed-files#7 and jitterbit/get-changed-files#11

jitterbit/get-changed-files#11 (comment)

latacora-tomekr mentioned this issue Nov 6, 2020

Add continue-on-error true to ignore head/base commit issue alex/nyt-2020-election-scraper#151

Merged

3 tasks

bcipriano mentioned this issue Jan 21, 2021

Add dockerignore so that the sandbox env can be run mutliple times. AcademySoftwareFoundation/OpenCue#879

Merged

nmelehan added a commit to nmelehan/docs that referenced this issue May 7, 2021

Integrate workaround for "head commit not ahead of base commit"

c551efc

Workaround proposed here: jitterbit/get-changed-files#11 (comment)

nmelehan mentioned this issue May 7, 2021

Vale GHA improvements linode/docs#4304

Merged

nmelehan added a commit to nmelehan/docs that referenced this issue May 7, 2021

Integrate workaround for "head commit not ahead of base commit"

4a253bf

Workaround proposed here: jitterbit/get-changed-files#11 (comment)

nmelehan mentioned this issue May 7, 2021

Testing out: Integrate workaround for "head commit not ahead of base commit" linode/docs#4305

Closed

sharkinsspatial added a commit to sharkinsspatial/staged-recipes that referenced this issue May 19, 2021

Handle base ordering issues refs jitterbit/get-changed-files#11

2e653a6

Ana06 mentioned this issue Jun 4, 2021

Remove response.data.status check #25

Closed

Ana06 mentioned this issue Jun 4, 2021

Head commit is not ahead of the base commit #19

Open

VictorSanh added a commit to bigscience-workshop/promptsource that referenced this issue Jun 8, 2021

workaround jitterbit error in GH actions taken from jitterbit/get-cha…

a8d2585

…nged-files#11

Xiol added a commit to ans-group/docs.ukfast.co.uk that referenced this issue Dec 6, 2021

Ignore errors in get-changed-files

1238d2f

When a set of commits is squashed and force pushed, it causes errors with this action. See jitterbit/get-changed-files#11

thekaveman mentioned this issue Dec 13, 2021

Fix nbviewer workflow cal-itp/data-analyses#36

Merged

trask mentioned this issue Jan 13, 2022

Fix failure when PR is behind main dyladan/component-owners#10

Merged

mdegat01 mentioned this issue Feb 9, 2022

Using CAS for content-trust home-assistant/supervisor#3382

Merged

12 tasks

ueda-yu added a commit to ueda-yu/UedaVer_template that referenced this issue May 24, 2022

Update action.yml

48797a2

Change to follow jitterbit/get-changed-files#11 and use uses: Ana06/get-changed-files@v1.2

koomie added a commit to pecos/tps that referenced this issue Jun 1, 2022

borrowing work-around for analyzing forced pushes

66b4569

(jitterbit/get-changed-files#11)

koomie added a commit to pecos/tps that referenced this issue Jun 2, 2022

borrowing work-around for analyzing forced pushes

ddba77f

(jitterbit/get-changed-files#11)

jmv74211 added a commit to wazuh/wazuh-qa that referenced this issue Jul 20, 2022

refac: Update workflow to get the PR updated files

8816fc3

The previous one seems to have had a bug reported here jitterbit/get-changed-files#11

jmv74211 added a commit to wazuh/wazuh-qa that referenced this issue Jul 20, 2022

refac: Update workflow to get the PR updated files

402d3c4

The previous one seems to have had a bug reported here jitterbit/get-changed-files#11

jmv74211 added a commit to wazuh/wazuh-qa that referenced this issue Jul 20, 2022

refac: Update workflow to get the PR updated files

02c0e66

The previous one seems to have had a bug reported here jitterbit/get-changed-files#11

jaimehgb mentioned this issue Aug 25, 2022

feat(#minor); CI/CD; Add capability to deploy only and add action to test builds on pushing to the remote branched in this repo. messari/subgraphs#879

Merged

dot2dotseurat mentioned this issue Oct 27, 2022

Making nft tables' trade_category more consistent duneanalytics/spellbook#1841

Merged

13 tasks

developerfred added a commit to developerfred/spellbook that referenced this issue Nov 18, 2022

chore(actions): hot fix jitterbit/get-changed-files#11

117b817

This was referenced Dec 20, 2022

Use Ana06/get-changed-files instead of jitterbit/get-changed-files ohpc-openeuler/ohpc#90

Closed

Use Ana06/get-changed-files instead of jitterbit/get-changed-files openhpc/ohpc#1640

Merged

marcusrbrown added a commit to bfra-me/ha-addon-repository that referenced this issue Jan 29, 2023

fix: replace the get-changed-files action with @Ana06's fork

b082801

See jitterbit/get-changed-files#11. This action appears to be unmaintained and is superceded by Ana06/get-changed-files.

marcusrbrown mentioned this issue Jan 29, 2023

fix: replace the get-changed-files action with @Ana06's fork bfra-me/ha-addon-repository#21

Merged

maltekliemann mentioned this issue Feb 3, 2023

Add automatic copyright notice verification zeitgeistpm/zeitgeist#963

Merged

hugovk added a commit to hugovk/cpython that referenced this issue Mar 7, 2023

Use Ana06/get-changed-files to fix 'The head commit for this pull_req…

de1e80b

…uest event is not ahead of the base commit' Re: jitterbit/get-changed-files#11

hugovk added a commit to hugovk/cpython that referenced this issue Mar 7, 2023

Use Ana06/get-changed-files to fix 'The head commit for this pull_req…

238dfc2

…uest event is not ahead of the base commit' Re: jitterbit/get-changed-files#11

FelipeLema mentioned this issue Aug 11, 2023

run tests only on files under source directories smoka7/hop.nvim#26

Merged

igorpeshansky mentioned this issue Sep 26, 2023

Make git change detection in PR presubmits more robust. GoogleCloudPlatform/ops-agent#1443

Merged

9 tasks

kopax-polyconseil mentioned this issue Jan 15, 2024

(BSR) ci: fix generate api template pass-culture/pass-culture-main#10226

Merged

3 tasks

joscha mentioned this issue May 14, 2024

fix(types): dedupe unique and greatest strategy needs id PipedreamHQ/pipedream#11945

Merged

joscha added a commit to joscha/pipedream that referenced this issue May 14, 2024

chore: workaround for changed files check due to forcce push

70e85af

see jitterbit/get-changed-files#54, jitterbit/get-changed-files#4, jitterbit/get-changed-files#7 and jitterbit/get-changed-files#11

joscha mentioned this issue May 14, 2024

chore: replace unmaintained jitterbit/get-changed-files@v1 PipedreamHQ/pipedream#11949

Merged

TomK added a commit to TomK/hassio-addon-transmission that referenced this issue Aug 20, 2024

skip errors in get-changed-files

d23d76a

jitterbit/get-changed-files#11 (comment)

TomK mentioned this issue Aug 20, 2024

skip errors in get-changed-files maorcc/hassio-addon-transmission#23

Open

penalosa mentioned this issue Oct 21, 2024

Fix head check for PR validation cloudflare/workers-sdk#7044

Merged

9 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

The head commit for this pull_request event is not ahead of the base commit #11

The head commit for this pull_request event is not ahead of the base commit #11

nolanbconaway commented Aug 18, 2020

Bertg commented Aug 25, 2020

justyn commented Sep 7, 2020

mmalek-sa commented Oct 28, 2020

ahoereth commented Oct 28, 2020 •

edited

Loading

johentsch commented Nov 28, 2020

dlam commented Dec 12, 2020

javipolo commented Dec 28, 2020

Ana06 commented Jun 4, 2021

vmartinv commented Feb 12, 2022

sgibson91 commented Apr 13, 2022

Bertg commented Apr 15, 2022

The head commit for this pull_request event is not ahead of the base commit #11

The head commit for this pull_request event is not ahead of the base commit #11

Comments

nolanbconaway commented Aug 18, 2020

Bertg commented Aug 25, 2020

justyn commented Sep 7, 2020

mmalek-sa commented Oct 28, 2020

ahoereth commented Oct 28, 2020 • edited Loading

johentsch commented Nov 28, 2020

dlam commented Dec 12, 2020

javipolo commented Dec 28, 2020

Ana06 commented Jun 4, 2021

vmartinv commented Feb 12, 2022

sgibson91 commented Apr 13, 2022

Bertg commented Apr 15, 2022

ahoereth commented Oct 28, 2020 •

edited

Loading