Skip to content

Cancel superseded CI runs so Pages deploys stay in order - #183

Merged
yyolk merged 1 commit into
masterfrom
cursor/ci-concurrency-cancel-92c2
Sep 6, 2026
Merged

Cancel superseded CI runs so Pages deploys stay in order#183
yyolk merged 1 commit into
masterfrom
cursor/ci-concurrency-cancel-92c2

Conversation

@yyolk

@yyolk yyolk commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Why

Two master CI runs overlapped: older #180 serial pages still running while #181 matrix pages started. Without concurrency, the older deploy can finish last and overwrite the newer catalog.

Change

Add GitHub Actions concurrency to .github/workflows/ci.yml (after on:, before jobs:):

concurrency:
  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
  cancel-in-progress: true

One group per PR number or branch ref; newer runs cancel superseded ones so Pages cannot deploy out of order.

publish.yml and release-pdfs.yml are unchanged. No job logic changes.

Open in Web Open in Cursor 

Co-authored-by: Joseph Chiocchi <joe@yolk.cc>
@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 068101b5-1b94-4e69-9f26-6f5899d15e65

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@yyolk
yyolk marked this pull request as ready for review September 6, 2026 06:15
@yyolk
yyolk merged commit 197e038 into master Sep 6, 2026
6 checks passed
@yyolk
yyolk deleted the cursor/ci-concurrency-cancel-92c2 branch September 6, 2026 06:15
@yyolk yyolk mentioned this pull request Sep 6, 2026
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