Skip to content

Run the test matrix once per PR commit - #444

Merged
alejandroschuler merged 1 commit into
masterfrom
claude/serene-bose-b722bd
Oct 27, 2026
Merged

alejandroschuler merged 1 commit into
masterfrom
claude/serene-bose-b722bd

Conversation

@alejandroschuler

Copy link
Copy Markdown
Collaborator
  • The push trigger now runs only on master. Before, each commit on a PR branch in this repo started the 6-version matrix twice, once for push and once for pull_request. Those extra jobs filled the runners, and the Docs deploys on master waited more than 20 minutes for one.
  • A new concurrency group is keyed on the PR number. A new push to a PR cancels that PR's older run.
  • Pushes to master get a group of their own (the run id), so a master run is never cancelled, even while it waits for a runner.
  • Note: a branch with no PR, or a PR whose base is not master, gets no CI from this workflow now.

🤖 Generated with Claude Code

The workflow ran on push to any branch and on pull_request to master, so
each commit on a same-repo PR branch ran the 6-version matrix twice. Push
now triggers only on master.

A concurrency group keyed on the PR number cancels the older run when a
PR gets a new push. Pushes to master fall back to the run id, so each
master run has its own group and is never cancelled, even while pending.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@alejandroschuler
alejandroschuler merged commit 662aae3 into master Oct 27, 2026
8 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