fix: pin dependency-audit reusable workflow to SHA#219
fix: pin dependency-audit reusable workflow to SHA#219
Conversation
Pins the reusable workflow reference from @v1 to commit SHA to satisfy the Action Pinning Policy (ci-standards.md#action-pinning-policy). SHA ee22b427cbce9ecadcf2b436acb57c3adf0cb63d corresponds to the v1 tag of petry-projects/.github as of 2026-04-19. Closes #158 Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
This PR is ready for review and merge. @don-petry — you are the code owner, could you please review and merge this to close the compliance finding in issue #158? This is a single-line change pinning the reusable workflow SHA. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated the reusable GitHub Actions workflow reference in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 dependency-audit reusable workflow reference to a specific commit SHA to satisfy the organization’s GitHub Actions pinning policy (closes #158).
Changes:
- Replace
@v1with a commit SHA forpetry-projects/.github/.github/workflows/dependency-audit-reusable.yml. - Add an inline
# v1note to preserve the semantic version context.
| jobs: | ||
| dependency-audit: | ||
| uses: petry-projects/.github/.github/workflows/dependency-audit-reusable.yml@v1 | ||
| uses: petry-projects/.github/.github/workflows/dependency-audit-reusable.yml@ee22b427cbce9ecadcf2b436acb57c3adf0cb63d # v1 |
There was a problem hiding this comment.
The header comment says you "MUST NOT change ... the uses: line", but this PR necessarily changes it to comply with the action pinning policy. Consider updating that guidance to reflect the intended maintenance process (e.g., uses should be pinned to a commit SHA, and only the SHA may be updated when the reusable workflow/tag advances) so future edits aren’t blocked by contradictory instructions.



Summary
@v1to commit SHAee22b427cbce9ecadcf2b436acb57c3adf0cb63dto satisfy the Action Pinning Policy.Before:
After:
The SHA was resolved from the
v1tag viagh api repos/petry-projects/.github/git/refs/tags/v1.Closes #158
Generated with Claude Code
Summary by CodeRabbit