Skip to content

Conversation

@yamadashy
Copy link
Owner

Skip Claude Code review when PR is in draft state to avoid unnecessary reviews while work is still in progress.

Checklist

  • Run npm run test
  • Run npm run lint

Add condition to skip code review when PR is in draft state.
This prevents unnecessary reviews while work is still in progress.
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 9, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

A condition is added to the Claude Code Review GitHub Actions workflow to skip processing when pull requests are marked as draft. The check is integrated into the existing conditional guard alongside repository and author validations.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
\.github/workflows/claude-code-review\.yml
Adds a draft PR skip condition (github.event.pull_request.draft == false) to the existing if block; repository match and non-Bot author checks remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/skip-draft-pr-review

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2e8f1c1 and 4039606.

📒 Files selected for processing (1)
  • .github/workflows/claude-code-review.yml (1 hunks)

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.

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.38%. Comparing base (2e8f1c1) to head (4039606).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #999   +/-   ##
=======================================
  Coverage   90.38%   90.38%           
=======================================
  Files         110      110           
  Lines        7890     7890           
  Branches     1528     1528           
=======================================
  Hits         7131     7131           
  Misses        759      759           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yamadashy yamadashy merged commit 50580c2 into main Dec 9, 2025
51 of 54 checks passed
@yamadashy yamadashy deleted the chore/skip-draft-pr-review branch December 9, 2025 15:27
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