adam/.github
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/.github.git synced 2026-09-30 15:13:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #108 from Sea-Haven-Industries/fix/reusable-concurrency-job-key

Browse source 
fix(ci): key multi-job reusable concurrency on a literal job id
This commit is contained in:
Adam Moussa 2026-07-28 17:26:19 -04:00 • committed by GitHub
commit b14028eeef
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 21 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
.github/workflows/ci-mobile-ios.yaml vendored
View file

@ -122,10 +122,16 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: ${{ inputs.js-timeout-minutes }}
# cancel-in-progress is TRUE: superseding a push should abandon the older
# CI run, which produces no external side effects. `github.job` is in the
# key so the three jobs here do not serialise against each other.
# CI run, which produces no external side effects.
#
# The trailing segment is the job id written out literally, NOT
# `${{ github.job }}`. In a called workflow that expression evaluates to the
# CALLER's job id, so every job here would resolve to the same group and,
# with cancel-in-progress on, cancel its own siblings. Observed live in
# pr-reviewer: `lint` was cancelled one second in by a sibling and the
# aggregator failed on the cancelled dependency.
concurrency:
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-${{ github.job }}
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-js
cancel-in-progress: true
defaults:
run:
@ -194,7 +200,7 @@ jobs:
runs-on: macos-26
timeout-minutes: ${{ inputs.ios-timeout-minutes }}
concurrency:
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-${{ github.job }}
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-ios-build
cancel-in-progress: true
defaults:
run:
@ -289,7 +295,7 @@ jobs:
if: always()
runs-on: ubuntu-latest
concurrency:
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-${{ github.job }}
group: ci-mobile-ios-${{ github.workflow }}-${{ github.ref }}-${{ inputs.working-directory }}-ci
cancel-in-progress: true
steps:
- name: Require all jobs to have succeeded

14
.github/workflows/ci-python-app.yaml vendored
View file

@ -50,8 +50,14 @@ jobs:
lint:
runs-on: ubuntu-latest
timeout-minutes: 10
# The trailing segment of every group in this file is the job id written
# out literally, NOT `${{ github.job }}`. In a called workflow that
# expression evaluates to the CALLER's job id, so all four jobs here would
# resolve to one group and, with cancel-in-progress on, cancel each other.
# Observed live in pr-reviewer: `lint` was cancelled one second in by a
# sibling and the aggregator failed on the cancelled dependency.
concurrency:
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-${{ github.job }}
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-lint
cancel-in-progress: true
steps:
- uses: actions/checkout@v7
@ -100,7 +106,7 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 10
concurrency:
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-${{ github.job }}
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-test-collect
cancel-in-progress: true
steps:
- uses: actions/checkout@v7
@ -124,7 +130,7 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 10
concurrency:
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-${{ github.job }}
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-subproject-tests
cancel-in-progress: true
steps:
- uses: actions/checkout@v7
@ -150,7 +156,7 @@ jobs:
if: always()
runs-on: ubuntu-latest
concurrency:
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-${{ github.job }}
group: ci-python-app-${{ github.workflow }}-${{ github.ref }}-${{ inputs.source-dirs }}-ci
cancel-in-progress: true
steps:
- name: Require all jobs to have succeeded