Add checks/statuses read permissions to Claude workflow#268
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 |
Review: Add checks/statuses read permissions to Claude workflowOverall: Clean, minimal, well-scoped change. The additions are correct and appropriate. A few observations: What is good
Minor observationThe pre-existing checks: read # Required for Claude to read CI check results
statuses: read # Required for Claude to read legacy commit statusesPre-existing note (out of scope for this PR)The job-level permissions use |
| checks: read | ||
| statuses: read |
There was a problem hiding this comment.
Consider adding inline comments for consistency with the existing actions: read annotation above:
| checks: read | |
| statuses: read | |
| checks: read # Required for Claude to read CI check results | |
| statuses: read # Required for Claude to read legacy commit statuses |
Greptile SummaryThis PR expands GitHub Actions token permissions to allow the Claude Code action to read CI check results and statuses on pull requests. Changes made:
The changes are minimal, consistent, and follow standard GitHub Actions patterns. All added permissions are read-only, presenting no security risk. Confidence Score: 5/5
Important Files Changed
Last reviewed commit: f4c4962 |
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 workflow-only change that expands the GitHub Actions token permissions to read
checks/statusesand passesgithub_tokeninto the Claude action; potential impact is limited to CI access scope.Overview
Updates the
Claude CodeGitHub Actions workflow to let the Claude action read CI results.The job now includes
checks: readandstatuses: readpermissions (and mirrors them inadditional_permissions), and explicitly passesgithub_token: ${{ github.token }}toanthropics/claude-code-action@v1.Written by Cursor Bugbot for commit f4c4962. Configure here.