Skip to content

ci: add concurrency groups to cancel stale runs - #21

Merged
acamarata merged 1 commit into
mainfrom
ci/consolidate-hygiene-p1
Sep 1, 2026
Merged

acamarata merged 1 commit into
mainfrom
ci/consolidate-hygiene-p1

Conversation

@acamarata

Copy link
Copy Markdown
Contributor

Summary

  • ntv is a fully retired product — this is intentionally the minimum useful change, not a rework.
  • Adds concurrency (cancel-in-progress) to dependency-audit.yml and generated-file-gate.yml so a follow-up push cancels an in-flight run instead of queuing for a second runner.
  • dependabot-auto-merge.yml left untouched (pull_request_targetgithub.ref resolves to the base branch, so a ref-keyed group would cross-cancel unrelated PRs).
  • No consolidation: generated-file-gate is the only workflow under the trivial-gate bar in this repo, and there's nothing else to fold it into.
  • No paths: filters added — repo is small enough that a missed run isn't worth risking for the marginal savings.

Safety

  • Required check name pnpm audit (job name in dependency-audit.yml) is unchanged.
  • No checks removed, no continue-on-error, no loosened assertions.

Test plan

  • actionlint on both touched files: exit 0, matches baseline (origin/main also exit 0)

Adds cancel-in-progress concurrency to dependency-audit and generated-file-gate.
ntv is a retired product so this is intentionally the minimum useful change:
no consolidation attempted (generated-file-gate is the only trivial workflow,
nothing else to fold it into) and no path filters added. dependabot-auto-merge
is left alone (pull_request_target trap — ref resolves to base branch).

Required check name 'pnpm audit' (job name in dependency-audit.yml) is unchanged.
@acamarata
acamarata merged commit 243ca67 into main Sep 1, 2026
3 checks passed
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.

1 participant