FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix race condition with actions/checkout@v1 by simon-engledew · Pull Request #428 · github/codeql-action · GitHub

Fix race condition with actions/checkout@v1 - #428

Merged
simon-engledew merged 3 commits into
mainfrom
simon-engledew/detect-merge
Mar 23, 2021
Merged

simon-engledew merged 3 commits into
mainfrom
simon-engledew/detect-merge

Conversation

simon-engledew commented Mar 22, 2021
edited
Loading

Copy link
Copy Markdown

Work around a race condition in Actions where sometimes GITHUB_SHA != git rev-parse head after an actions/checkout@v1

This leads to the upload incorrectly rewriting the ref, causing analyses to go missing. 😭

Do not rewrite the ref when we:

  • are on GITHUB_SHA
  • are on GITHUB_REF

simon-engledew force-pushed the simon-engledew/detect-merge branch 3 times, most recently from 69dbd1b to 4c8a765 Compare March 22, 2021 11:40
simon-engledew changed the title Count the number of parents of the current commit to check it is still a merge Fix race condition with actions/checkout@v1 where GITHUB_SHA != git rev-parse HEAD Mar 22, 2021
simon-engledew changed the title Fix race condition with actions/checkout@v1 where GITHUB_SHA != git rev-parse HEAD Fix race condition with actions/checkout@v1 Mar 22, 2021
simon-engledew force-pushed the simon-engledew/detect-merge branch from 4c8a765 to b0ed899 Compare March 22, 2021 12:01
…l a merge

Work around a race condition in actions where sometimes GITHUB_SHA != git rev-parse head
simon-engledew force-pushed the simon-engledew/detect-merge branch from b0ed899 to ef92c5a Compare March 22, 2021 12:05
simon-engledew marked this pull request as ready for review March 22, 2021 12:43
Comment thread src/actions-util.ts Outdated
Comment thread src/actions-util.ts Outdated
Comment thread src/actions-util.ts
simon-engledew merged commit 972dc3e into main Mar 23, 2021
simon-engledew deleted the simon-engledew/detect-merge branch March 23, 2021 06:18
simon-engledew pushed a commit that referenced this pull request Mar 25, 2021
Fixes the rev-parse issues caused by #428
github-actions Bot mentioned this pull request Mar 26, 2021
aofaof0907 pushed a commit to aofaof0907/codeql-action that referenced this pull request Jul 29, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL