ci: path-based skip and gate job in pr_checks#3615
Conversation
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (30)
WalkthroughThis PR refactors Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
pr_checksruns the full matrix on every PR. #3609 touched onlyapps/webapp/app/routes/admin.tsxand still ran the 4-job CLI e2e matrix and 5-job sdk-compat suite.Adds a
changesjob usingdorny/paths-filterand gates each tier:apps/webapp/**,packages/**,internal-packages/**packages/**internal-packages/**+packages/**(cross-deps)packages/{cli-v3,build,core,schema-to-json}/**packages/{trigger-sdk,core}/**.configs/**,package.json,pnpm-lock.yaml,pnpm-workspace.yaml,turbo.jsonare also included in every filter since they affect the whole workspace.Inlines the
unitsreusable-workflow children so each can be gated independently (status check names also flatten fromunits / webapp / ...towebapp / ...).unit-tests.ymlis unaffected - still used bypublish.yml.Adds an
all-checksgate that always runs and short-circuits to success when every dependent is success-or-skipped. With this in place a single required status check (All PR Checks) is enough; before this,paths-ignorewould have left required checks Pending on docs/changeset PRs (gh docs).