Skip to content

Commit 53fc995

Browse files
Run CI on all main/postsubmit commits.
Prior to this, the concurrency rules for presubmits (which cancel eagerly) were being applied to main. The result was that landing a second patch would cancel the CI on the one prior.
1 parent 2148c4c commit 53fc995

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

.github/workflows/bazelBuildAndTest.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,11 @@ on:
1111
# any in-progress jobs in the same github workflow and github
1212
# ref (e.g. refs/heads/main or refs/pull/<pr_number>/merge).
1313
concurrency:
14-
group: ${{ github.workflow }}-${{ github.ref }}
14+
# A PR number if a pull request and otherwise the commit hash. This cancels
15+
# queued and in-progress runs for the same PR (presubmit) or commit
16+
# (postsubmit). The workflow name is prepended to avoid conflicts between
17+
# different workflows.
18+
group: ${{ github.workflow }}-${{ github.event.number || github.sha }}
1519
cancel-in-progress: true
1620

1721

.github/workflows/buildAndTest.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,11 @@ on:
1212
# any in-progress jobs in the same github workflow and github
1313
# ref (e.g. refs/heads/main or refs/pull/<pr_number>/merge).
1414
concurrency:
15-
group: ${{ github.workflow }}-${{ github.ref }}
15+
# A PR number if a pull request and otherwise the commit hash. This cancels
16+
# queued and in-progress runs for the same PR (presubmit) or commit
17+
# (postsubmit). The workflow name is prepended to avoid conflicts between
18+
# different workflows.
19+
group: ${{ github.workflow }}-${{ github.event.number || github.sha }}
1620
cancel-in-progress: true
1721

1822

0 commit comments

Comments
 (0)