Merged
Conversation
Apply five wins to reduce CI wall-clock time, especially on Windows: 1. Add `concurrency` groups to both test workflows so stale PR runs are cancelled when a new commit is pushed. 2. Rebalance Windows acceptance shards (`a-l` → `a-e` + `f-l`) so the slowest shard drops from ~2m20s to ~1m10s. 3. Add `paths-ignore` filter so pushes touching only `docs/**`, `adrs/**`, `*.md` do not trigger the full test matrix. 4. Split Windows acceptance jobs into `windows-acceptance` running only on `push` to `main` — keeps coverage on merge, removes the long acceptance wait from every PR. 5. Pass `--jobs 4` to `./bashunit --parallel` on Windows to match the 4-core runner and avoid over-subscription on Git Bash. Also gitignore `.claude/scheduled_tasks.lock`.
Contributor
✅ Contributor ReportUser: @Chemaclass This user is on the trusted contributors list and was automatically approved. |
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
1 participant
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
Five wins to cut CI wall-clock time, especially on Windows where Git Bash process spawn is ~10x slower than Linux:
tests.yml+tests-bash-3.0.yml— cancel stale in-flight runs as soon as a new commit is pushed to a PR.a-l(~2m20s) split intoa-e+f-l(~1m10s each) — 10 test files per shard.paths-ignorefilter — skip full test matrix when onlydocs/**,adrs/**,*.md, issue/PR templates change.push: main, not on every PR — Windows unit + functional still run on PRs; acceptance still validated before merge hits main.--jobs 4on Windows matches the 4-core runner and avoids oversubscription on Git Bash.Also ignore
.claude/scheduled_tasks.lock(local Claude Code state).Test plan
ruby -ryamlvalidates both YAML filespush: main(after merge) runs the 3 acceptance shards