Repository navigation
Correct the pr_review.py Quota-Refusal Remedy Comment - #1832
Conversation
The comment above the refusal reader said a quota refusal is cleared by waiting, contradicting the module docstring's exit-code contract for 46: a quota refusal is account-level state that no wait clears, so the caller proceeds on the other reviewers' coverage instead. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ptr727
added
the
comments
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 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 |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The change is comment-only and correctly aligns the inline guidance with the existing documented exit-code behavior.
Review effort: Lite
Findings: None
What changed in this PR
Updates scripts/pr_review.py to correct an inline comment about how to respond to Copilot review refusals, aligning it with the script's documented exit-code contract (quota refusals are not cleared by waiting).
Changes:
- Rewords the refusal-remedy comment to separate file-count refusals (split PR) from quota refusals (account-level; proceed using other reviewers' coverage).
| File | Description |
|---|---|
| scripts/pr_review.py | Clarifies the quota-refusal remedy in the digest output comment without changing behavior. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
gh pr create attaches a label with a separate mutation after opening the pull request, so the opened event's payload carried no label and validate-task.yml's allow-comments input read false. A re-run of that same run reuses its original event payload and fails the same way. This empty commit's synchronize event reads the label already on the pull request. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
… Fixes to Main (#1839) Promotes `develop` to `main`. ## Carried - #1830: stops the prose gate reading a `#` after `$` or `${` as a comment marker, per #1646. - #1832: corrects the comment on the `pr_review.py` quota-refusal remedy, per #1647. - #1834: accepts an absent, empty, or wildcard SessionEnd matcher as covering every exit reason, per #1638. - #1838: routes every question and every offer of more work through the interface's prompt mechanism, blocking or not. - #1841: describes a SessionEnd matcher by its shape in the installer report, following #1834. Closes #1646 Closes #1647 Closes #1638 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Fixes the comment
pr_review.py's refusal reader carries, per #1647.What changes
The inline comment above the refusal reader said a quota refusal is cleared by waiting, which
contradicted the module docstring's own exit-code contract for
46: a quota refusal isaccount-level state that no re-request or wait clears, so the caller proceeds on the other
reviewers' coverage instead. The comment now says that. Behavior and exit codes are unchanged,
this is a comment-only fix.
Verification
ruff checkpasses on the changed file. The local strict review pass found no issues, recordedagainst develop. The prose gate's
comment-addedrule fires on this edit as expected, since itrewords an existing comment line, and this PR carries the
commentslabel to stand it down.Closes on promotion: #1647
🤖 Generated with Claude Code