fix: Bug in the timer command help message

This commit is contained in:
Jeremy Wall 2023-08-23 19:19:09 -04:00
parent e980306761
commit fb954c6f0d

View File

@ -14,9 +14,9 @@
// runwhen - A utility that runs commands on user defined triggers. // runwhen - A utility that runs commands on user defined triggers.
#[macro_use] #[macro_use]
extern crate clap; extern crate clap;
extern crate glob;
extern crate humantime; extern crate humantime;
extern crate notify; extern crate notify;
extern crate glob;
use std::{process, str::FromStr}; use std::{process, str::FromStr};
@ -58,7 +58,7 @@ fn do_flags() -> clap::ArgMatches {
clap::Command::new("timer") clap::Command::new("timer")
.about("Run command on a timer") .about("Run command on a timer")
.arg(arg!(-t --duration).takes_value(true).value_parser(value_parser!(humantime::Duration)).help("Duration between runs")) .arg(arg!(-t --duration).takes_value(true).value_parser(value_parser!(humantime::Duration)).help("Duration between runs"))
.arg(arg!(-n --repeat).value_parser(value_parser!(u32))).about("Number of times to run before finishing")) .arg(arg!(-n --repeat).value_parser(value_parser!(u32)).help("Number of times to run before finishing")))
.subcommand( .subcommand(
clap::Command::new("success") clap::Command::new("success")
.about("Run a command when a test command succeeds") .about("Run a command when a test command succeeds")
@ -80,7 +80,7 @@ fn main() {
} }
maybe_env = Some(env_vec); maybe_env = Some(env_vec);
} }
let mut proc: Box<dyn Process> = if let Some(matches) = app.subcommand_matches("watch") { let mut proc: Box<dyn Process> = if let Some(matches) = app.subcommand_matches("watch") {
let file = match matches.values_of("file") { let file = match matches.values_of("file") {
Some(v) => v.collect(), Some(v) => v.collect(),
@ -100,7 +100,9 @@ fn main() {
None => None, None => None,
}; };
println!("Enforcing a poll time of {:?}", duration); println!("Enforcing a poll time of {:?}", duration);
Box::new(FileProcess::new(cmd, maybe_env, file, exclude, method, duration)) Box::new(FileProcess::new(
cmd, maybe_env, file, exclude, method, duration,
))
} else if let Some(matches) = app.subcommand_matches("timer") { } else if let Some(matches) = app.subcommand_matches("timer") {
// TODO(jwall): This should use cancelable commands. // TODO(jwall): This should use cancelable commands.
// Unwrap because this flag is required. // Unwrap because this flag is required.