From 6f215eee98c4f73099b0ede2ac62ba019ada24ce Mon Sep 17 00:00:00 2001 From: Gibson Fahnestock Date: Thu, 8 Aug 2024 13:58:34 +0100 Subject: [PATCH] refactor(styles): make styles example use a const This makes it easier to copy this example for use in the derive API, like so: ```rust const STYLES: Styles = Styles::styled() .header(AnsiColor::Green.on_default().bold()) .usage(AnsiColor::Green.on_default().bold()) .literal(AnsiColor::Blue.on_default().bold()) .placeholder(AnsiColor::Cyan.on_default()); #[derive(Parser)] #[clap(styles = STYLES)] struct Cmd { ... } ``` If you use the `|` method then it's not a constant function. --- Cargo.lock | 6 +++--- clap_builder/Cargo.toml | 2 +- clap_builder/src/builder/command.rs | 10 +++++----- src/bin/stdio-fixture.rs | 10 +++++----- 4 files changed, 14 insertions(+), 14 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index a04c3626..a8d9b857 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -101,9 +101,9 @@ dependencies = [ [[package]] name = "anstyle" -version = "1.0.7" +version = "1.0.8" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "038dfcf04a5feb68e9c60b21c9625a54c2c0616e79b72b0fd87075a056ae1d1b" +checksum = "1bec1de6f59aedf83baf9ff929c98f2ad654b97c9510f4e70cf6f661d49fd5b1" [[package]] name = "anstyle-parse" @@ -1432,7 +1432,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "e310b3a6b5907f99202fcdb4960ff45b93735d7c7d96b760fcff8db2dc0e103d" dependencies = [ "cfg-if", - "windows-targets 0.48.5", + "windows-targets 0.52.6", ] [[package]] diff --git a/clap_builder/Cargo.toml b/clap_builder/Cargo.toml index 827c31bc..afb3f05a 100644 --- a/clap_builder/Cargo.toml +++ b/clap_builder/Cargo.toml @@ -63,7 +63,7 @@ clap_lex = { path = "../clap_lex", version = "0.7.0" } unicase = { version = "2.6.0", optional = true } strsim = { version = "0.11.0", optional = true } anstream = { version = "0.6.7", optional = true } -anstyle = "1.0.0" +anstyle = "1.0.8" terminal_size = { version = "0.3.0", optional = true } backtrace = { version = "0.3.73", optional = true } unicode-width = { version = "0.1.9", optional = true } diff --git a/clap_builder/src/builder/command.rs b/clap_builder/src/builder/command.rs index 37d5802e..6480e28b 100644 --- a/clap_builder/src/builder/command.rs +++ b/clap_builder/src/builder/command.rs @@ -1199,13 +1199,13 @@ impl Command { /// ```no_run /// # use clap_builder as clap; /// # use clap::{Command, ColorChoice, builder::styling}; - /// let styles = styling::Styles::styled() - /// .header(styling::AnsiColor::Green.on_default() | styling::Effects::BOLD) - /// .usage(styling::AnsiColor::Green.on_default() | styling::Effects::BOLD) - /// .literal(styling::AnsiColor::Blue.on_default() | styling::Effects::BOLD) + /// const STYLES: styling::Styles = styling::Styles::styled() + /// .header(styling::AnsiColor::Green.on_default().bold()) + /// .usage(styling::AnsiColor::Green.on_default().bold()) + /// .literal(styling::AnsiColor::Blue.on_default().bold()) /// .placeholder(styling::AnsiColor::Cyan.on_default()); /// Command::new("myprog") - /// .styles(styles) + /// .styles(STYLES) /// .get_matches(); /// ``` #[cfg(feature = "color")] diff --git a/src/bin/stdio-fixture.rs b/src/bin/stdio-fixture.rs index efd6c817..03ecd1fd 100644 --- a/src/bin/stdio-fixture.rs +++ b/src/bin/stdio-fixture.rs @@ -15,12 +15,12 @@ fn main() { #[cfg(feature = "color")] { use clap::builder::styling; - let styles = styling::Styles::styled() - .header(styling::AnsiColor::Green.on_default() | styling::Effects::BOLD) - .usage(styling::AnsiColor::Green.on_default() | styling::Effects::BOLD) - .literal(styling::AnsiColor::Blue.on_default() | styling::Effects::BOLD) + const STYLES: styling::Styles = styling::Styles::styled() + .header(styling::AnsiColor::Green.on_default().bold()) + .usage(styling::AnsiColor::Green.on_default().bold()) + .literal(styling::AnsiColor::Blue.on_default().bold()) .placeholder(styling::AnsiColor::Cyan.on_default()); - cmd = cmd.styles(styles); + cmd = cmd.styles(STYLES); } cmd.get_matches(); }