From 51e03621035b96ac2999ed764a3c305f03ba4258 Mon Sep 17 00:00:00 2001 From: Robert Chang Date: Sun, 24 Mar 2024 04:23:25 +0800 Subject: [PATCH] Remove the tracing::field::debug method of is_a_git_repo function [WHY] Because of the command check the destination_path is a git repo or not only contains two situation. One is yes, another one is not. so we don't need to consider it as a error and display to the user, they will be confused. [HOW] 1. Remove it --- loco-cli/src/git.rs | 4 ---- 1 file changed, 4 deletions(-) diff --git a/loco-cli/src/git.rs b/loco-cli/src/git.rs index 2ae1c9657..373c8c8ce 100644 --- a/loco-cli/src/git.rs +++ b/loco-cli/src/git.rs @@ -159,10 +159,6 @@ pub fn is_a_git_repo(destination_path: &Path) -> eyre::Result { if output.status.success() { Ok(true) } else { - tracing::error!( - error = tracing::field::debug(output), - "git command returned an error" - ); Ok(false) } }