error out if lint name contains dash

JarredAllen · May 23, 2023 · a3438da · a3438da
1 parent 435a8ad
commit a3438da
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/clippy_dev/src/main.rs b/clippy_dev/src/main.rs
@@ -180,7 +180,14 @@ fn get_clap_config() -> ArgMatches {
                         .short('n')
                         .long("name")
                         .help("Name of the new lint in snake case, ex: fn_too_long")
-                        .required(true),
+                        .required(true)
+                        .value_parser(|name: &str| {
+                            if name.contains('-') {
+                                Err("Lint name cannot contain `-`, use `_` instead.")
+                            } else {
+                                Ok(name.to_owned())
+                            }
+                        }),
                     Arg::new("category")
                         .short('c')
                         .long("category")