fix: pin dependency-audit reusable workflow to SHA#182
fix: pin dependency-audit reusable workflow to SHA#182
Conversation
Pin petry-projects/.github reusable workflow to commit SHA instead of mutable @v1 tag to satisfy action-pinning policy. Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 45 minutes and 35 seconds. ⌛ 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: Path: .coderabbit.yaml 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 |
|
There was a problem hiding this comment.
Pull request overview
Pins the org-level reusable workflow reference in dependency-audit.yml to an immutable commit SHA to satisfy the action pinning compliance requirement (Issue #158).
Changes:
- Updated
uses: petry-projects/.github/...@v1to@ae9709f...(with an inline# v1note).
| jobs: | ||
| dependency-audit: | ||
| uses: petry-projects/.github/.github/workflows/dependency-audit-reusable.yml@v1 | ||
| uses: petry-projects/.github/.github/workflows/dependency-audit-reusable.yml@ae9709f4466dec60a5733c9e7487f69dcd004e05 # v1 |
There was a problem hiding this comment.
The header comment above says you "MUST NOT change" the uses: line, but this workflow now correctly updates uses: to a pinned SHA for compliance. Please update the header guidance to reflect that uses: should remain pinned to the approved SHA (and only be updated when syncing from the source-of-truth), otherwise future editors may revert this back to a mutable tag.
Automated review — APPROVEDRisk: LOW SummarySingle-line change pins a reusable workflow reference from the mutable @v1 tag to its resolved commit SHA (ae9709f4), satisfying the org action-pinning policy (closes #158). The SHA is confirmed correct via the GitHub API — it is exactly the commit the annotated v1 tag points to. All CI checks pass (CodeQL, SonarCloud, build, tests, AgentShield) with zero new issues. FindingsInfo
Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with |



Summary
petry-projects/.githubreusable workflow reference independency-audit.ymlfrom the mutable@v1tag to the resolved commit SHAae9709f4466dec60a5733c9e7487f69dcd004e05(v1)Closes #158
Generated with Claude Code