Repository navigation
ci: report filtered-out test shards under their real names - #215
Conversation
A skipped matrix job reports as 'Go tests (${{ matrix.shard }})', so a
web-only PR never produced the required 'Go tests (http)'/'(rest)'
checks (and a Go-only PR would never produce 'Web tests (n/4)'), leaving
it blocked. Stub jobs run exactly when the real shards are filtered out
and report the expanded names.
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe CI workflow adds matrix stub jobs for Go and web tests. When the respective path filters report no changes, these jobs emit passing messages for each shard. ChangesCI matrix checks
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The change adds passing stub jobs with the expanded check names when Go or web shards are filtered out, which unblocks required checks. No actionable merge risk remains. Architecture SummaryArchitecture risk: 🔵 Low · up to The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency. Changed systems: None identified. Architecture concerns Review detailsBefore / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Branch protection requires
Go tests (http),Go tests (rest)andWeb tests (1..4/4). When paths-filter skips a matrix job, GitHub reports it under the unexpanded name (Go tests (${{ matrix.shard }})), so a web-only PR (e.g. #214) never gets the Go shard checks and stays blocked, and vice versa.Adds
go-test-full-skipped/web-test-coverage-skippedstubs that run only when the real shards are filtered out and report the same expanded names.🤖 Generated with Claude Code
Summary by CodeRabbit