🥅 Enhance error handling for dirty tree

Before, when the tree was checked for being dirty, the error message was duplicated around the code.
Now, it is a registered known error

Author: julien040

src/controller/error.go

@@ -16,7 +16,7 @@ var (
 		Err:     errors.New("error reading input"),
 	}
 	errorWorkingTreeNotClean = GutError{
-		Message: "I can't continue further because the working tree is not clean",
+		Message: "I can't continue further because the working tree is not clean. Commit your changes with gut save",
 		Code:    2,
 		Err:     errors.New("working tree not clean"),
 	}
@@ -56,17 +56,21 @@ func getLinkForError(GutError GutError) string {
 //
 // This should be used when the error is known and can be resolved by the user
 func exitOnKnownError(typeOfError GutError, err error) {
+
+	// When the error is linked to the user input, we don't print the error message
+	// because it's not useful
+	if typeOfError.Code == 1 {
+		os.Exit(1)
+		return
+	}
 	// Print the error message to stderr
 	fmt.Fprintln(os.Stderr, "")
 	if err != nil {
 		fmt.Fprintf(os.Stderr, color.RedString("%s (error: %s)\n"), color.RedString(typeOfError.Message), color.RedString(err.Error()))
-		/* print.Message("%s (error: %s)", print.Error, typeOfError.Message, err.Error()) */
 	} else {
 		fmt.Fprintf(os.Stderr, "%s\n", color.RedString(typeOfError.Message))
-		/* print.Message("%s", print.Error, typeOfError.Message) */
 	}
 	fmt.Fprintf(os.Stderr, "To resolve this issue, please follow the instructions on this page: %s\n", getLinkForError(typeOfError))
-	/* print.Message("To resolve this issue, please follow the instructions on this page: %s", print.Optional, getLinkForError(typeOfError)) */
 
 	os.Exit(1)
 }

src/controller/merge.go

@@ -27,7 +27,7 @@ func Merge(cmd *cobra.Command, args []string) {
 		exitOnError("Sorry, I can't check if the working directory is clean 😢", err)
 	}
 	if !clean {
-		exitOnError("Uh oh, there are uncommitted changes. Please commit them before merging branches 😢", nil)
+		exitOnKnownError(errorWorkingTreeNotClean, nil)
 
 	}
 

src/controller/switch.go

@@ -58,7 +58,7 @@ func Switch(cmd *cobra.Command, args []string) {
 			}
 			// If not clean, we return an error
 			if !clean {
-				exitOnError("Sorry, you have uncommitted changes. Save them with \"gut save\" before going to another commit or you will lose them", nil)
+				exitOnKnownError(errorWorkingTreeNotClean, nil)
 			}
 
 			// Switch to the commit

src/controller/sync.go

@@ -34,7 +34,7 @@ func Sync(cmd *cobra.Command, args []string) {
 		exitOnError("Sorry, I can't check if there are uncommited changes 😢", err)
 	}
 	if !clean {
-		exitOnError("Uh oh, there are uncommited changes. Please commit them before syncing with `gut save`", nil)
+		exitOnKnownError(errorWorkingTreeNotClean, nil)
 	}
 
 	// Get the remote