Mirrors/rust-clippy
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-clippy synced 2024-11-10 15:14:29 +00:00
Code Issues Projects Releases Packages Wiki Activity

Make --explain subcommand return 1 for missing lints

Browse source
This commit is contained in:
Martin Fischer 2023-06-16 11:11:20 +02:00
parent a94aac607c
commit 894d5dafac
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
clippy_lints/src/lib.rs
View file

@ -495,7 +495,7 @@ pub(crate) struct LintInfo {
explanation: &'static str,
}
pub fn explain(name: &str) {
pub fn explain(name: &str) -> i32 {
let target = format!("clippy::{}", name.to_ascii_uppercase());
if let Some(info) = declared_lints::LINTS.iter().find(|info| info.lint.name == target) {
println!("{}", info.explanation);
@ -512,8 +512,10 @@ pub fn explain(name: &str) {
println!(" - {}: {} (default: {})", conf.name, conf.doc, conf.default);
}
}
0
} else {
println!("unknown lint: {name}");
1
}
}

4
src/main.rs
View file

@ -57,7 +57,9 @@ pub fn main() {
if let Some(pos) = env::args().position(|a| a == "--explain") {
if let Some(mut lint) = env::args().nth(pos + 1) {
lint.make_ascii_lowercase();
clippy_lints::explain(&lint.strip_prefix("clippy::").unwrap_or(&lint).replace('-', "_"));
process::exit(clippy_lints::explain(
&lint.strip_prefix("clippy::").unwrap_or(&lint).replace('-', "_"),
));
} else {
show_help();
}