-
Notifications
You must be signed in to change notification settings - Fork 147
Add a command to synchronize advisory data from osv.dev/GHSA #656
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
f5aadee
to
f5a07a8
Compare
cargo-audit/src/commands/audit.rs
Outdated
@@ -1,5 +1,7 @@ | |||
//! The `cargo audit` subcommand | |||
|
|||
#![allow(unused_qualifications)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rust 1.70.0 produces:
warning: unnecessary qualification
--> cargo-audit/src/commands/audit.rs:38:12
|
38 | #[clap(subcommand)]
| ^^^^^^^^^^
|
note: the lint level is defined here
--> cargo-audit/src/lib.rs:21:56
|
21 | #![warn(missing_docs, rust_2018_idioms, trivial_casts, unused_qualifications)]
| ^^^^^^^^^^^^^^^^^^^^^
I did not investigate it further for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really need to upgrade clap
(and abscissa
)
c10e285
to
c90b9e7
Compare
d464bc8
to
f56fb9b
Compare
* tame_index usage * license addition to the advisories * New advisory sources in external IDs in GHSA
@amousset any reason not to merge this and start running it on CI? It would be great to have this up and running! |
It's been running correctly since a long time, I think it makes sense to start automating this indeed! |
Result in rustsec/advisory-db#1693.
Refs: #644
This PR implements:
The end goal is to be able to automatically open pull requests with the advisories updates and drafts from missing ones. For now I plan to run it regularly manually until we're confident with the result.