ci: SHA-pin third-party actions in #325 bundle workflows (closes #331) - #340
ci: SHA-pin third-party actions in #325 bundle workflows (closes #331)#340robotrocketscience wants to merge 6 commits into
Conversation
Pin step-security/harden-runner@v2.19.0 and crate-ci/typos@v1.46.0 (was @master) to 40-char commit SHAs.
Pin step-security/harden-runner@v2.19.0 and astral-sh/setup-uv@v5.4.2 to 40-char commit SHAs.
Pin step-security/harden-runner@v2.19.0 and astral-sh/setup-uv@v5.4.2 to 40-char commit SHAs.
Pin step-security/harden-runner@v2.19.0 and astral-sh/setup-uv@v5.4.2 to 40-char commit SHAs.
Pin step-security/harden-runner@v2.19.0, lycheeverse/lychee-action@v2.8.0, and peter-evans/create-issue-from-file@v5.0.1 to 40-char commit SHAs.
Pin step-security/harden-runner@v2.19.0 to 40-char commit SHA. github/codeql-action/* are first-party and remain on @V3.
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ 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 (6)
✨ 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. Review rate limit: 0/1 reviews remaining, refill in 44 minutes and 43 seconds.Comment |
Reviewer's guide (collapsed on small PRs)Reviewer's GuidePins all third-party GitHub Actions used in the six bundle workflows introduced in #325 to specific commit SHAs (while keeping tags as comments), replacing floating version tags and master refs to improve supply-chain security and reproducibility. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Since the same SHAs (e.g.,
step-security/harden-runnerandastral-sh/setup-uv) are repeated across multiple workflows, consider centralizing them via a reusable workflow, composite action, or YAML anchors so updating pins in the future is less error-prone.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Since the same SHAs (e.g., `step-security/harden-runner` and `astral-sh/setup-uv`) are repeated across multiple workflows, consider centralizing them via a reusable workflow, composite action, or YAML anchors so updating pins in the future is less error-prone.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
Superseded by #338. Identical pin choices across all 6 workflow files (verified via |
Summary
Closes #331. SHA-pin every third-party action invocation in the 6 bundle workflows added by #325. Each pinned line keeps a
# <tag>trailing comment for human readability.Pins
crate-ci/typosbbaefadf97b0ec5fdc942684b647f1a6ab250274step-security/harden-runner8d3c67de8e2fe68ef647c8db1e6a09f647780f40astral-sh/setup-uvd4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86lycheeverse/lychee-action8646ba30535128ac92d33dfc9133794bfdd9b411peter-evans/create-issue-from-filee8ef132d6df98ed982188e460ebb3b5d4ef3a9cdFirst-party
actions/*(checkout, upload-artifact, github/codeql-action/*) left as@vNper GitHub guidance.Anchoring
Each pin resolves to the latest release within the workflow's current major (e.g. setup-uv pinned to v5.4.2 even though v8.1.0 exists). Major bumps are a separate decision and out of scope for this sweep.
Verification
crate-ci/typos@masteris gone.Closes #331.
Summary by Sourcery
CI: