Skip to content

ci: add concurrency-cancellation groups to workflows - #452

Merged
DaveSkender merged 1 commit into
mainfrom
fac-375-add-concurrency-groups
Jul 17, 2026
Merged

ci: add concurrency-cancellation groups to workflows#452
DaveSkender merged 1 commit into
mainfrom
fac-375-add-concurrency-groups

Conversation

@facioquobo

@facioquobo facioquobo Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Superseded workflow runs — e.g. rapid pushes to the same branch or PR — currently run to completion instead of being cancelled, consuming CI minutes for results that are already obsolete.

This adds a standard GitHub Actions concurrency: group to the workflows that were missing one, keyed on ${{ github.workflow }}-${{ github.head_ref || github.ref }}:

  • Build / test workflows (ci.yml, test-indicators.yml, test-indicators-full.yml, test-localization.yml, test-performance.yml) use cancel-in-progress: true, so a newer run cancels the prior in-progress one.
  • deploy-package.yml (PyPI publish) uses cancel-in-progress: false, so an in-flight release is never interrupted mid-run — the group key still coalesces duplicate triggers.

Additive-only: no existing job, step, trigger, cache, or artifact behavior is changed.

Test plan

  • All modified workflow files parse as valid YAML locally.
  • CI on this PR exercises the affected workflows; pushing a follow-up commit to this branch demonstrates the earlier run transitioning to cancelled.

Superseded workflow runs (rapid pushes to the same ref or PR) currently run
to completion, wasting Actions minutes. Add a concurrency group keyed on the
workflow name plus head_ref/ref so a newer run cancels the prior in-progress
one. The deploy-package publish workflow uses cancel-in-progress: false so an
in-flight PyPI release is never interrupted mid-run.
@facioquobo
facioquobo Bot marked this pull request as ready for review July 17, 2026 02:22
@facioquobo
facioquobo Bot requested a review from a team July 17, 2026 02:22
@facioquoqa

facioquoqa Bot commented Jul 17, 2026

Copy link
Copy Markdown

:neckbeard: FaciōQuōqa Mage automatic code review is disabled in your mage.yml

  • Trigger review

@DaveSkender
DaveSkender merged commit d527aae into main Jul 17, 2026
16 checks passed
@DaveSkender
DaveSkender deleted the fac-375-add-concurrency-groups branch July 17, 2026 02:37
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Stock Indicators for Python Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants