ci(repo): trigger CI on ready_for_review#8467
Conversation
Draft-gated jobs (build, tests, static analysis) skip when github.event.pull_request.draft is true. By default, pull_request only fires on opened/synchronize/reopened, so converting a draft PR to ready-for-review never re-runs CI and the gated jobs stay skipped. Adding ready_for_review to the trigger list ensures CI re-evaluates and the gated jobs run.
🦋 Changeset detectedLatest commit: d90da48 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis pull request adds configuration to the GitHub Actions CI workflow to restrict pull request event triggers to specific types: Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Review rate limit: 4/5 reviews remaining, refill in 12 minutes. Comment |
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
Summary
Draft-gated jobs in
.github/workflows/ci.yml(build, tests, static analysis, dedupe/changeset) checkgithub.event.pull_request.draft == falseand skip when the PR is a draft. By default,pull_requestonly fires onopened,synchronize, andreopened— converting a draft PR to ready-for-review does not retrigger the workflow, so the gated jobs stay skipped until someone pushes a new commit.Adding
ready_for_reviewtopull_request.typesmakes CI re-evaluate when the PR exits draft, so all the gated jobs run without needing a no-op push.Context
Hit on #8466 — PR was opened as draft, CI fired and skipped the heavy jobs, then the PR was marked ready-for-review and never re-ran CI.
Test plan