fix(check): Use Cargo's colors

With `rustup check`s colors being testable (#4561),
this updates it to match Cargo's colors

Author: epage

src/cli/rustup_mode.rs

@@ -11,10 +11,10 @@ use std::{
 };
 
 use anstream::ColorChoice;
-use anstyle::{AnsiColor, Style};
+use anstyle::Style;
 use anyhow::{Context, Error, Result, anyhow};
 use clap::{Args, CommandFactory, Parser, Subcommand, ValueEnum, builder::PossibleValue};
-use clap_cargo::style::{CONTEXT, HEADER};
+use clap_cargo::style::{CONTEXT, ERROR, GOOD, HEADER, TRANSIENT, WARN};
 use clap_complete::Shell;
 use futures_util::stream::StreamExt;
 use indicatif::{MultiProgress, ProgressBar, ProgressStyle};
@@ -825,23 +825,13 @@ async fn check_updates(cfg: &Cfg<'_>, opts: CheckOpts) -> Result<ExitCode> {
         Style::new()
     };
 
-    let error = if use_colors {
-        AnsiColor::Red.on_default().bold()
-    } else {
-        Style::new()
-    };
+    let transient = if use_colors { TRANSIENT } else { Style::new() };
 
-    let good = if use_colors {
-        AnsiColor::Green.on_default().bold()
-    } else {
-        Style::new()
-    };
+    let error = if use_colors { ERROR } else { Style::new() };
 
-    let warn = if use_colors {
-        AnsiColor::Yellow.on_default().bold()
-    } else {
-        Style::new()
-    };
+    let good = if use_colors { GOOD } else { Style::new() };
+
+    let warn = if use_colors { WARN } else { Style::new() };
 
     // Ensure that `.buffered()` is never called with 0 as this will cause a hang.
     // See: https://github.com/rust-lang/futures-rs/pull/1194#discussion_r209501774
@@ -852,7 +842,9 @@ async fn check_updates(cfg: &Cfg<'_>, opts: CheckOpts) -> Result<ExitCode> {
         let channels = tokio_stream::iter(channels.into_iter()).map(|(name, distributable)| {
             let msg = format!("{bold}{name} - {bold:#}");
             let status = "Checking...";
-            let template = format!("{{msg}}{status} {{spinner:.green}}");
+            let template = format!(
+                "{{msg}}{transient}{status}{transient:#} {transient}{{spinner}}{transient:#}"
+            );
             let pb = multi_progress_bars.add(ProgressBar::new(1));
             pb.set_style(
                 ProgressStyle::with_template(template.as_str())

src/cli/self_update.rs

@@ -40,11 +40,12 @@ use std::process::Command;
 use std::str::FromStr;
 use std::{fmt, fs};
 
-use anstyle::{AnsiColor, Style};
+use anstyle::Style;
 use anyhow::{Context, Result, anyhow};
 use cfg_if::cfg_if;
 use clap::ValueEnum;
 use clap::builder::PossibleValue;
+use clap_cargo::style::{GOOD, WARN};
 use itertools::Itertools;
 use same_file::Handle;
 use serde::{Deserialize, Serialize};
@@ -1398,8 +1399,8 @@ pub(crate) async fn check_rustup_update(dl_cfg: &DownloadCfg<'_>) -> Result<bool
     let available_version = get_available_rustup_version(dl_cfg).await?;
 
     let bold = Style::new().bold();
-    let yellow = AnsiColor::Yellow.on_default().bold();
-    let green = AnsiColor::Green.on_default().bold();
+    let yellow = WARN;
+    let green = GOOD;
 
     write!(t, "{bold}rustup - {bold:#}")?;