Skip to content

Commit 41ae646

Browse files
eraobi1kenobi
andauthored
Running cargo semver-checks with no other arguments should default to checking (#459)
* cargo semver-checks with no other arguments should default to checking * Update tests/feature_config.rs Co-authored-by: Predrag Gruevski <[email protected]> * addresses comments on PR * cargo fmt * Update src/main.rs Co-authored-by: Predrag Gruevski <[email protected]> * address comments from pr * trailing line between constant and method --------- Co-authored-by: Predrag Gruevski <[email protected]>
1 parent 817b1ff commit 41ae646

File tree

2 files changed

+140
-64
lines changed

2 files changed

+140
-64
lines changed

src/main.rs

+12-16
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ fn main() -> anyhow::Result<()> {
4848
)?;
4949
}
5050

51-
let mut config = GlobalConfig::new().set_level(args.verbosity.log_level());
51+
let mut config = GlobalConfig::new().set_level(args.check_release.verbosity.log_level());
5252
config.shell_note("Use `--explain <id>` to see more details")?;
5353
std::process::exit(0);
5454
} else if let Some(id) = args.explain.as_deref() {
@@ -75,19 +75,15 @@ fn main() -> anyhow::Result<()> {
7575
std::process::exit(0);
7676
}
7777

78-
match args.command {
79-
Some(SemverChecksCommands::CheckRelease(args)) => {
80-
let check: cargo_semver_checks::Check = args.into();
81-
let report = check.check_release()?;
82-
if report.success() {
83-
std::process::exit(0)
84-
} else {
85-
std::process::exit(1);
86-
}
87-
}
88-
None => {
89-
anyhow::bail!("subcommand required");
90-
}
78+
let check: cargo_semver_checks::Check = match args.command {
79+
Some(SemverChecksCommands::CheckRelease(args)) => args.into(),
80+
None => args.check_release.into(),
81+
};
82+
let report = check.check_release()?;
83+
if report.success() {
84+
std::process::exit(0);
85+
} else {
86+
std::process::exit(1);
9187
}
9288
}
9389

@@ -111,8 +107,8 @@ struct SemverChecks {
111107
#[arg(long, global = true, exclusive = true)]
112108
list: bool,
113109

114-
#[command(flatten)]
115-
verbosity: clap_verbosity_flag::Verbosity<clap_verbosity_flag::InfoLevel>,
110+
#[clap(flatten)]
111+
check_release: CheckRelease,
116112

117113
#[command(subcommand)]
118114
command: Option<SemverChecksCommands>,

0 commit comments

Comments
 (0)