Skip to content

ci: skip checks for draft PRs#1274

Merged
omeraplak merged 1 commit into
VoltAgent:mainfrom
zrosenbauer:chore/ci-drafts
May 11, 2026
Merged

ci: skip checks for draft PRs#1274
omeraplak merged 1 commit into
VoltAgent:mainfrom
zrosenbauer:chore/ci-drafts

Conversation

@zrosenbauer
Copy link
Copy Markdown
Contributor

@zrosenbauer zrosenbauer commented May 8, 2026

Summary

  • Skip PR check jobs while a pull request is marked as a draft.
  • Run the PR checks when the pull request is marked ready for review.
  • Add an empty changeset for the CI-only change.

Why

CI is currently running on draft PRs, which wastes GitHub Actions minutes and is unnecessary while work is still in draft state.

Validation

  • Parsed .github/workflows/pull-request.yml as YAML.
  • Ran git diff --check.

Summary by cubic

Skip PR check jobs while a PR is in draft and run them when it’s marked ready for review. This reduces unnecessary GitHub Actions minutes on WIP PRs.

Written for commit 15a457d. Summary will update on new commits.

Summary by CodeRabbit

Release Notes

  • Chores
    • Draft pull requests now skip automated checks to streamline the development workflow.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 8, 2026

🦋 Changeset detected

Latest commit: 15a457d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When 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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 8, 2026

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a54d994c-d50d-4c56-a6ff-4eeead8aed39

📥 Commits

Reviewing files that changed from the base of the PR and between ad4c44c and 15a457d.

📒 Files selected for processing (2)
  • .changeset/quiet-draft-pr-checks.md
  • .github/workflows/pull-request.yml

📝 Walkthrough

Walkthrough

The pull request configures GitHub Actions to skip CI check jobs when a pull request is in draft state. The workflow trigger is updated to listen for specific PR event types, and six jobs are guarded with draft-state conditions to prevent execution on draft PRs.

Changes

Draft PR Check Skipping

Layer / File(s) Summary
Workflow Trigger Configuration
.github/workflows/pull-request.yml
Workflow trigger is narrowed from generic pull_request to specific event types (opened, synchronize, reopened, ready_for_review) to enable draft-state detection.
Job-Level Draft Guards
.github/workflows/pull-request.yml
Six jobs (commit-lint, lint, build, test-packages, postgres-integration, e2e-tests) are each updated with if: ${{ !github.event.pull_request.draft }} to skip execution when a PR is in draft state.
Changeset Documentation
.changeset/quiet-draft-pr-checks.md
Changeset entry documents the feature to skip PR checks for draft pull requests.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

A draft PR floats in gentle breeze,
No checks shall run to cause unease—
The workflow pauses, wise and kind,
Till author's work is well-designed,
Then ready state unlocks the gates! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: skipping CI checks for draft pull requests.
Description check ✅ Passed The description covers the main changes, rationale, and validation, though it deviates from the template structure with auto-generated content.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@zrosenbauer
Copy link
Copy Markdown
Contributor Author

@omeraplak not sure if you prefer CI on for drafts but figured it could save minutes and at least on my end I like pushing out a first version to possibly get eyes on or iterate a bit, I see that coderabbit is ignoring drafts already...

@omeraplak omeraplak merged commit 2fc23a9 into VoltAgent:main May 11, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants