ci: optimize PR pipeline — concurrency, fast gate, reduced matrix#215
Merged
RtlZeroMemory merged 1 commit intomainfrom Feb 26, 2026
Merged
ci: optimize PR pipeline — concurrency, fast gate, reduced matrix#215RtlZeroMemory merged 1 commit intomainfrom
RtlZeroMemory merged 1 commit intomainfrom
Conversation
- Add concurrency group to ci.yml and codeql.yml to cancel in-progress runs on re-push (biggest win for runner contention) - Extract lint/typecheck/codegen/portability/unicode into a dedicated `checks` job that gates the matrix — lint failures caught in ~2 min instead of after the full 15-min pipeline - Dynamic matrix: 5 runners on PRs (Linux × Node 18/20/22, macOS × 22, Windows × 22), full 3×3 on push to main - Remove redundant lint/typecheck/codegen/biome-install steps from each matrix cell and the bun job - Remove duplicate docs job (already handled by docs.yml) Net effect on PRs: 13 jobs → 9, ~44% fewer runners, fast-fail on static checks, stale runs cancelled automatically. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe CI workflow is refactored to introduce concurrency controls, extract fast platform-independent checks into a dedicated job, and compute test matrices dynamically based on event type. The CodeQL workflow gains concurrency configuration to cancel redundant PR runs. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ci.ymlandcodeql.yml— cancel in-progress runs when a new push arrives on the same PR, freeing runners immediatelychecksgate — lint, typecheck, codegen, portability, and unicode checks run once on Linux/Node 22 (~2 min); failures prevent the expensive matrix from starting at alldocsjob removed (already indocs.yml)Before vs After (PR)
Full 3×3 matrix still runs on push to main — no safety regression.
Test plan
docsrequired status check doesn't reference the removed ci.yml job (if applicable in branch protection)🤖 Generated with Claude Code
Summary by CodeRabbit