ci: skip typecheck for refs and non-code file PRs#3624
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)
📜 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). (33)
🔇 Additional comments (1)
WalkthroughThis PR updates the GitHub Actions workflow configuration for PR checks by adding three path exclusion filters to the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 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 |
Follow-up to #3615. The
codefilter currently fires typecheck for any change outsidedocs/,.changeset/,hosting/, or.github/- so a docs-only PR like #3623 (touchingreferences/ai-chat/.env.example+README.md) triggered the typecheck job. None of thereferences/*packages declare atypecheckscript either, so even when a real code change lands there,turbo run typecheckskips them. Running the job is pure cost.Tightens the filter to also exclude:
references/**- playground projects, none of them contribute toturbo run typechecktoday**/*.md- markdown anywhere**/.env.example- example env files anywhereTwo known gaps left open:
typecheckscripts to each (or top-leveltsc -p), or accept playground status.changesjob still runs (it's a path-filter step) but the dependent jobs all skip on irrelevant PRs.