Skip to content

fix: enable Claude code review bot to approve clean PRs#370

Merged
ninja-shreyash merged 2 commits intomainfrom
fix/claude-review-approval
Apr 16, 2026
Merged

fix: enable Claude code review bot to approve clean PRs#370
ninja-shreyash merged 2 commits intomainfrom
fix/claude-review-approval

Conversation

@ninja-shreyash
Copy link
Copy Markdown
Contributor

Summary

  • Adds Step 6 to the review workflow prompt instructing Claude to gh pr review --approve when no issues are found
  • Adds Bash(gh pr review:*) to the allowed tools list so the bot can execute the approval command
  • Previously, the bot only ever left COMMENTED reviews (79 across 20 PRs) and never approved

Test plan

  • Tested on ninja-shreyash/cherrypick-test#22 — bot approved a clean PR with state APPROVED
  • Verify existing comment behavior is unaffected on a PR with real issues

🤖 Generated with Claude Code

Add Step 6 to the review workflow prompt instructing Claude to approve
PRs when no issues are found. Also add `gh pr review` to the allowed
tools list so the bot can actually run the approval command.

Tested on ninja-shreyash/cherrypick-test#22 — bot successfully approved
a clean PR with state APPROVED.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ninja-shreyash ninja-shreyash requested a review from a team April 14, 2026 18:31
@vnaren23
Copy link
Copy Markdown
Collaborator

Will this count as a review count? 🤔

@ninja-shreyash
Copy link
Copy Markdown
Contributor Author

Will this count as a review count? 🤔

yes it will count .. I just tested

@ninja-shreyash ninja-shreyash force-pushed the fix/claude-review-approval branch from b9a8dc5 to 40bd0cf Compare April 16, 2026 03:54
@sonarqubecloud
Copy link
Copy Markdown

@ninja-shreyash ninja-shreyash merged commit c04d7f9 into main Apr 16, 2026
14 of 15 checks passed
@ninja-shreyash ninja-shreyash deleted the fix/claude-review-approval branch April 16, 2026 05:07
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.

3 participants