Skip to content

ci: check Rust dependencies with cargo-deny - #76

Merged
velocitysystems merged 5 commits into
silvermine:masterfrom
gabref:ci/cargo-deny
Nov 3, 2026
Merged

velocitysystems merged 5 commits into
silvermine:masterfrom
gabref:ci/cargo-deny

Conversation

@gabref

@gabref gabref commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

CI currently does not check Rust dependencies for advisories or enforce license and source policies. Add a dedicated cargo-deny job on pushes and pull requests, installing pinned cargo-deny 0.20.2 and using the Rust toolchain declared in rust-toolchain.toml.

The policy checks all platforms and development dependencies, rejects wildcard dependencies and unapproved sources, and reports duplicate versions as warnings. Update affected dependencies, raise the Tokio and time minimum versions, replace the yanked spin version, and document two upstream exceptions: proc-macro-error and glib in Tauri's GTK stack.

Closes #74

Update plist, anyhow, and event-listener, and replace yanked spin.
Require patched Tokio and time versions for downstream consumers.
Check advisories, licenses, duplicate versions, and sources on pushes,
pull requests, and weekly. Document unavoidable GTK exceptions.
@gabref
gabref requested a review from a team October 27, 2026 09:23
Comment thread Cargo.toml Resolved
Comment thread .github/workflows/ci.yml Resolved
Comment thread .github/workflows/ci.yml Resolved
Comment thread .github/workflows/ci.yml Resolved
Comment thread .github/workflows/ci.yml Resolved

@velocitysystems velocitysystems left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please squash the review follow-up commits into the commits they amend before merge:

  • fix: refresh example Tokio lockfiles into fix: update vulnerable Rust dependencies
  • ci: run pinned cargo-deny directly on push and PR into ci: check Rust dependencies with cargo-deny

Comment thread README.md Resolved
Comment thread .github/workflows/ci.yml Resolved
@velocitysystems
velocitysystems merged commit 928cdd8 into silvermine:master Nov 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add cargo-deny to CI

2 participants