fix: move vouch/skip checks before diff acquisition#47
Conversation
Move --vouch and --skip handling to before any diff acquisition (PR fetch, file reads, git show/diff) so they short-circuit immediately after CLI args are parsed.
|
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 |
There was a problem hiding this comment.
Pull request overview
This PR optimizes the argus review command by moving --vouch and --skip flag checks to execute immediately after CLI argument parsing, before any diff acquisition operations (PR fetches, file reads, or git operations). This allows the command to short-circuit early and avoid unnecessary work when these flags are used.
Changes:
- Moved
--vouchand--skipchecks to execute before diff acquisition operations - Simplified the vouch output message from a computed format to a static string
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Move --vouch and --skip handling to before any diff acquisition (PR fetch, file reads, git show/diff) so they short-circuit immediately after CLI args are parsed.