fix: replace dtolnay/rust-toolchain with rustup in dependency-audit.yml#76
Conversation
Removes the unpinned external action dtolnay/rust-toolchain@stable and replaces it with the built-in rustup command as specified in the org standards template. Also syncs the file with the latest org template (adds pnpm support, updates cargo-audit to 0.22.1, updates checkout SHAs). Closes #47 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 3 minutes and 31 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)
Comment |
|
@don-petry — this PR is ready for your review and merge. |
|
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions dependency-audit workflow to align with org standards and address compliance finding #47 by removing an unpinned external Rust toolchain action.
Changes:
- Replace
dtolnay/rust-toolchain@stablewith arustup toolchain install stable --profile minimalstep. - Add pnpm ecosystem detection + a dedicated
pnpm auditjob. - Update pinned action SHAs and bump
cargo-auditto0.22.1.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - name: Audit Cargo dependencies | ||
| run: | |
There was a problem hiding this comment.
Inside the Audit Cargo dependencies script (below this run: |), the subshell uses cd "$dir" && cargo generate-lockfile ...; cargo audit—because of the ;, cargo audit will still run even if the cd fails (auditing the wrong directory). Consider chaining cargo audit with && (or otherwise explicitly handling failures) to ensure the audit only runs in the intended directory.



Summary
dtolnay/rust-toolchain@stable(compliance finding Compliance: unpinned-actions-dependency-audit.yml #47)rustup toolchain install stable --profile minimal— a built-in command that requires no external action and no SHA pinningdependency-audit.ymlwith the latest org standards template (petry-projects/.github/standards/workflows/dependency-audit.yml), which also adds pnpm audit support and updates cargo-audit to 0.22.1Closes #47
Generated with Claude Code