Don't emit an empty error string.

Allow matching the tool repo/alias as well as <author>/<repo> Use 'stable' as the Rust version.
reynn · Sep 14, 2023 · 0a0c7fa · 0a0c7fa
1 parent 682e5a9
commit 0a0c7fa
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/rust-toolchain.toml b/rust-toolchain.toml
@@ -1,4 +1,4 @@
 [toolchain]
-channel = "1.70"
+channel = "stable"
 components = ["rustfmt", "clippy"]
 profile = "minimal"
diff --git a/src/actions/update.rs b/src/actions/update.rs
@@ -31,7 +31,10 @@ pub async fn update_tools(
                 }
                 progress_bar.inc(1);
             }
-            error!("{}", failed_tools.join("\n"));
+
+            if !failed_tools.is_empty() {
+                error!("{}", failed_tools.join("\n"));
+            }
 
             Ok(())
         }
@@ -44,11 +47,15 @@ pub async fn update_tools(
             } else {
                 None
             };
-            if let Some(tool) = tools.iter().find(|t| t.name == split_name[0]) {
-                info!("Tool: {:?}", tool);
+
+            if let Some(tool) = tools
+                .iter()
+                .find(|t| t.name == split_name[0] || t.alias == split_name[0])
+            {
+                info!("Updating: {:?}", tool);
 
                 match super::handle_tool_install(env, tool, system, version).await {
-                    Ok(_) => info!("Tool {} has been updated.", &tool.name),
+                    Ok(_) => info!("{} has been updated.", &tool.name),
                     Err(install_err) => error!("{:?}", install_err),
                 }
             } else {