Skip to content

Add cargo deny check in CI#76

Merged
djc merged 2 commits intomainfrom
deny
Apr 1, 2025
Merged

Add cargo deny check in CI#76
djc merged 2 commits intomainfrom
deny

Conversation

@djc
Copy link
Member

@djc djc commented Mar 27, 2025

As discussed in rustls/rustls-ffi#554 (comment). This both checks the lockfile against the RustSec advisory database and checks dependency licenses as well as some other stuff. I've used this in many other projects (like Quinn), and Brian also set it up for webpki.

Copy link
Member

@cpu cpu left a comment

Choose a reason for hiding this comment

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

Thanks!

Nit-picking slightly, it might be nicer to separate the --locked commit & the Cargo.lock addition from the deny.toml & audit CI addition. Argh: I had clicked the wrong tab. You already did that 😆 Sorry!

@djc djc added this pull request to the merge queue Apr 1, 2025
Merged via the queue into main with commit b59e08d Apr 1, 2025
30 checks passed
@djc djc deleted the deny branch April 1, 2025 11:40
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.

3 participants