fix: pin dependabot-rebase reusable workflow to SHA (issue #157)#181
fix: pin dependabot-rebase reusable workflow to SHA (issue #157)#181
Conversation
|
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 51 minutes and 43 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 |
|
@don-petry This PR is ready for your review. It pins the reusable workflow reference to the commit SHA to resolve the action-pinning compliance finding (#157). |
|
There was a problem hiding this comment.
Pull request overview
Pins the Dependabot rebase reusable workflow reference to an immutable commit SHA to satisfy the org’s action-pinning compliance policy and clear issue #157.
Changes:
- Updated
.github/workflows/dependabot-rebase.ymlto use a full commit SHA instead of the mutable@v1tag for the reusable workflow.
| contents: read | ||
| pull-requests: read | ||
| uses: petry-projects/.github/.github/workflows/dependabot-rebase-reusable.yml@v1 | ||
| uses: petry-projects/.github/.github/workflows/dependabot-rebase-reusable.yml@ae9709f4466dec60a5733c9e7487f69dcd004e05 # v1 |
There was a problem hiding this comment.
The header comments say this workflow should be adopted verbatim and that the uses: line MUST NOT be changed, but this PR intentionally changes it for action-pinning compliance. Consider updating the nearby header guidance to explicitly allow/require pinning the reusable workflow to an immutable SHA (or note that this repo intentionally diverges from the current upstream template) to reduce the chance of a future revert back to @v1.
Automated review — APPROVEDRisk: LOW SummaryThis PR pins a reusable workflow reference from the mutable @v1 tag to the immutable commit SHA ae9709f4466dec60a5733c9e7487f69dcd004e05, resolving action-pinning compliance finding #157. The change is a pure security improvement (supply-chain hardening), 1 line in a single file, and every CI check — CodeQL, SonarCloud (0 new issues, 0 hotspots), AgentShield, dependency audit, and all test suites — passed. Triage escalated only due to a system-level triage output failure, not any content-level concern. FindingsInfo
CI statusAll status checks passed: CodeQL (actions/js-ts/python), SonarCloud Quality Gate, AgentShield, CI Pipeline, Node.js Tests, Playwright UI Tests, Coverage, Dependency audit. No blocking failures. 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 |
|
Superseded by #194 which adopts the full standard verbatim with the corrected SHA. Closing this stale PR. |



Summary
petry-projects/.github/.github/workflows/dependabot-rebase-reusable.ymlfrom the mutable tag@v1to the immutable commit SHAae9709f4466dec60a5733c9e7487f69dcd004e05 # v1action-pinningcompliance finding flagged by the weekly auditDetail
The compliance audit (
unpinned-actions-dependabot-rebase.yml) requires alluses:references to be pinned to a full commit SHA per the Action Pinning Policy.SHA was resolved by dereferencing the annotated
v1tag:Note: The upstream template at
petry-projects/.github/standards/workflows/dependabot-rebase.ymlstill uses@v1. A follow-up PR against the central repo would bring the template itself into compliance and prevent the finding from recurring in new repos.Closes #157
Generated with Claude Code