Add checks/statuses read permissions to Claude workflow#36
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ 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 |
PR Review: Add checks/statuses read permissions to Claude workflowThis is a clean, minimal, and low-risk change. Here's the analysis: What the PR does
Code Quality & Best Practices ✅The change follows GitHub Actions best practices well:
Security ✅No security concerns. All new permissions are read-only scopes on CI metadata:
These are appropriate for a workflow whose job is to let Claude understand CI context on PRs. The job already had Potential Issue: Duplicate
|
Applies the workflow update from shakacode/react_on_rails#2487:
checks: readandstatuses: readto job permissionsgithub_token: ${{ github.token }}toanthropics/claude-code-actionchecks: readandstatuses: readtoadditional_permissionsAdmin merge requested to keep rollout fast across repos.
Note
Low Risk
Low risk: limited to GitHub Actions permission/config changes for the Claude workflow, with no production code impact. The main consideration is the slightly expanded read-only token surface area for CI metadata.
Overview
Updates the
Claude CodeGitHub Actions workflow to let theanthropics/claude-code-actionread more CI metadata by grantingchecks: readandstatuses: readat the job level.Also passes
github_token: ${{ github.token }}into the action and mirrors the samechecks/statusesread scopes inadditional_permissionsso Claude can fetch check run and status context on PRs.Written by Cursor Bugbot for commit 6118797. Configure here.