ci: skip tests for doc-comment-only PRs#629
Conversation
When a PR only modifies doc comments (/// and //!) in .rs files without touching build files or actual code, skip the test suite and coverage. Linting and formatting still run. The detection works by diffing .rs files against the PR base, extracting all added/removed lines, and checking if every non-blank line starts with /// or //!. Build file changes (Cargo.toml, Cargo.lock) and push events always run the full suite. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
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. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 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 |
Summary
///,//!) in.rsfilesHow it works
The
detect-changesjob now has aneeds-testsoutput in addition toany-code:.toml,.lock) changed → tests always needed.rsfiles changed on a PR → diff against the base branch, extract all added/removed lines, check if every non-blank line starts with///or//!needs-tests=false, test jobs are skippedpull_requestevents)Test plan
🤖 Generated with Claude Code