minor CLI output tweaks

wenyuzhao · May 19, 2024 · f0825c9 · f0825c9
1 parent db89a55
commit f0825c9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/harness-cli/src/commands/run/runner.rs b/harness-cli/src/commands/run/runner.rs
@@ -272,7 +272,6 @@ impl<'a> BenchRunner<'a> {
         let csv_path = self.log_dir.as_ref().unwrap().join("results.csv");
         print_md!("Raw benchmark results at:\n");
         print_md!("* `{}`\n\n", csv_path.display());
-        print_md!("Please run `cargo harness report` to view results.\n");
     }
 
     fn get_inv_label(&self, index: usize, is_row_label: bool) -> String {

diff --git a/harness-cli/src/commands/upload/mod.rs b/harness-cli/src/commands/upload/mod.rs
@@ -1,10 +1,14 @@
 use std::path::PathBuf;
 
 use clap::Parser;
+use colored::Colorize;
 use reqwest::blocking::Client;
 use serde_json::{Map, Value};
 
-use crate::configs::run_info::{CrateInfo, RunInfo};
+use crate::{
+    configs::run_info::{CrateInfo, RunInfo},
+    print_md,
+};
 
 /// Upload benchmark results to https://r.harness.rs
 #[derive(Parser)]
@@ -77,8 +81,9 @@ impl UploadResultsArgs {
         if status.as_u16() != 201 {
             anyhow::bail!("Results already uploaded: {remote_url}");
         }
-        println!("Results uploaded: {remote_url}");
-        println!("Please claim the results by visiting the link above. Failure to claim the results within 7 days will result in automatic deletion.");
+        print!("{}", "✔ Benchmark results uploaded: ".green());
+        print_md!("`{remote_url}`\n");
+        print_md!("Please claim the results by visiting the link above. Failure to claim the results within 7 days will result in automatic deletion.\n");
         Ok(())
     }
 }