Skip to content

Commit

Permalink
fix(cli): remove --line-prefix shortcut
Browse files Browse the repository at this point in the history
--output=line-prefix does the same and --no-header has no effect on line-prefix
  • Loading branch information
EdJoPaTo committed Sep 6, 2024
1 parent 4d6850b commit e9b0648
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 16 deletions.
1 change: 0 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- `--no-header`
- `--output` with `inherit`, `line-prefix`, `collect` and `null`
- `--result` with `always`, `never` and `non-zero`
- `--line-prefix` as a shortcut for `--no-header --output=line-prefix`
- `--only-result` as a shortcut for `--no-header --output=null`
- `--quiet` as a shortcut for `--no-header --output=null --result=never`

Expand Down
15 changes: 0 additions & 15 deletions src/cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -139,17 +139,6 @@ pub struct Cli {
#[allow(clippy::option_option)]
threads: Option<Option<NonZeroUsize>>,

/// Shortcut for `--no-header --output=line-prefix`.
#[arg(
long,
short,
conflicts_with_all = ["no_header", "output"],
group = "command-output-shortcut",
requires = "command",
help_heading = "Command Option Shortcuts"
)]
line_prefix: bool,

/// Shortcut for `--no-header --result=never`.
#[arg(
long,
Expand Down Expand Up @@ -247,10 +236,6 @@ impl Cli {
eprintln!("project-below Hint: --relative is replaced by --path-style=working-dir and will be removed in the next major release");
matches.path_style = PathStyle::WorkingDir;
}
if matches.line_prefix {
matches.no_header = true;
matches.output = CommandOutput::LinePrefix;
}
if matches.no_harness {
matches.no_header = true;
matches.result = CommandResult::Never;
Expand Down

0 comments on commit e9b0648

Please sign in to comment.