Skip to content

Commit

Permalink
Update clap requirement from 3.2 to 4.0 (bevyengine#6303)
Browse files Browse the repository at this point in the history
# Objective

Alternative to bevyengine#6150

Dependabot's PR doesn't seem to break anything, but there are some deprecations that we might as well fix up.

## Solution

https://github.com/clap-rs/clap/blob/master/CHANGELOG.md#migrating

Update clap in `build-wasm-example` and `span-cmp`. Other tools don't use clap.

Remove references to `value_parser`. It's the default now.

Change `#[clap()]` to `#[arg()]`.

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
  • Loading branch information
2 people authored and Pietrek14 committed Dec 17, 2022
1 parent a1be424 commit 08f4cbb
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 11 deletions.
2 changes: 1 addition & 1 deletion tools/build-wasm-example/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ license = "MIT OR Apache-2.0"

[dependencies]
xshell = "0.2"
clap = { version = "3.2", features = ["derive"] }
clap = { version = "4.0", features = ["derive"] }
7 changes: 3 additions & 4 deletions tools/build-wasm-example/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,17 @@ use xshell::{cmd, Shell};
#[derive(Parser, Debug)]
struct Args {
/// Examples to build
#[clap(value_parser)]
examples: Vec<String>,

#[clap(short, long, value_parser)]
#[arg(short, long)]
/// Run tests
test: bool,

#[clap(short, long, value_parser)]
#[arg(short, long)]
/// Run on the given browsers. By default, chromium, firefox, webkit
browsers: Vec<String>,

#[clap(short, long, value_parser)]
#[arg(short, long)]
/// Stop after this number of frames
frames: Option<usize>,
}
Expand Down
2 changes: 1 addition & 1 deletion tools/spancmp/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ license = "MIT OR Apache-2.0"
[dependencies]
serde_json = "1.0"
serde = { version = "1.0", features = ["derive"] }
clap = { version = "3.2", features = ["derive"] }
clap = { version = "4.0", features = ["derive"] }
regex = "1.5"
termcolor = "1.1"
bevy_utils = { path = "../../crates/bevy_utils", version = "0.9.0-dev" }
Expand Down
8 changes: 3 additions & 5 deletions tools/spancmp/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,22 +15,20 @@ mod pretty;

#[derive(Parser, Debug)]
struct Args {
#[clap(short, long, value_parser, default_value_t = 0.0)]
#[arg(short, long, default_value_t = 0.0)]
/// Filter spans that have an average shorther than the threshold
threshold: f32,

#[clap(short, long, value_parser)]
#[arg(short, long)]
/// Filter spans by name matching the pattern
pattern: Option<Regex>,

#[clap(short, long, value_parser)]
#[arg(short, long)]
/// Simplify system names
short: bool,

#[clap(value_parser)]
trace: String,
/// Optional, second trace to compare
#[clap(value_parser)]
second_trace: Option<String>,
}

Expand Down

0 comments on commit 08f4cbb

Please sign in to comment.