fix(ci): make REUSE compliance check always run on PRs#49
Conversation
"Check REUSE Compliance" is a required status check, but reuse.yml had a paths filter whose "**/*" pattern does not match repository-root files (e.g. renovate.json). Root-only PRs therefore skipped the workflow, the required context never reported, and the PR was silently BLOCKED. Remove the paths filter so the required gate always runs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Review limit reached
More reviews will be available in 21 minutes and 57 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE 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 |
There was a problem hiding this comment.
Pull request overview
Ensures the required GitHub Actions status check Check REUSE Compliance always reports on pull requests by removing a paths filter that could skip the workflow on repo-root-only changes, leaving the required context pending.
Changes:
- Removed the
pull_request.pathsfilter from the REUSE compliance workflow so it runs on all PRs. - Added inline documentation explaining why the workflow must not be path-filtered for PRs.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Required check
Check REUSE Compliancewas gated behind a paths filter whose**/*does not match repo-root files (e.g.renovate.json), so root-only PRs (#46, #42) skipped the workflow and blocked silently. Removing the paths filter makes the required gate always report. Same class of issue as the #48 keystone.