💄 Fix printing issue on Windows cmd (fix #56)

CMD handles colors differently than other terminals. 
Therefore we need to use the color.Output from fatih/color
to ensure that the colors are printed correctly.

This commit rewrites all raw printing using fmt.Printf
to use the color.Output destination instead of os.Stdout.

Author: julien040

src/controller/clone.go

@@ -70,7 +70,7 @@ func Clone(cmd *cobra.Command, args []string) {
 	}
 
 	/* --------------------------------- Clone repo ------------------------------ */
-	fmt.Printf("\nYour repo is %s and will be cloned in %s", color.GreenString(repo), color.BlueString(path))
+	fmt.Fprintf(color.Output, "\nYour repo is %s and will be cloned in %s", color.GreenString(repo), color.BlueString(path))
 	s := spinner.New(spinner.CharSets[9], 100*time.Millisecond) // Build a new spinner
 	s.Start()
 	err = executor.Clone(repo, path, shouldConserveGitHistory)

src/controller/pathHelper_test.go

@@ -9,6 +9,8 @@ import (
 	"os"
 	"path/filepath"
 	"testing"
+
+	"github.com/fatih/color"
 )
 
 func Test_getAbsPathFromInput(t *testing.T) {
@@ -107,7 +109,7 @@ func Test_checkIfPathExist(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			if got := checkIfPathExist(tt.args.path); got != tt.want {
-				fmt.Printf("Path: %s", tt.args.path)
+				fmt.Fprintf(color.Output, "Path: %s", tt.args.path)
 				t.Errorf("checkIfPathExist() = %v, want %v", got, tt.want)
 			}
 		})

src/controller/profile.go

@@ -269,11 +269,11 @@ func selectProfile(gitURL string, createPossible bool) profile.Profile {
 func Profiles(cmd *cobra.Command, args []string) {
 	print.Message("Print info about the profiles \n", print.Info)
 	profileSelected := selectProfile("", true)
-	fmt.Printf(color.HiBlackString("Profile selected: %s \n"), color.HiBlueString(profileSelected.Alias))
-	fmt.Printf(color.HiBlackString("Email: %s \n"), color.HiBlueString(profileSelected.Email))
-	fmt.Printf(color.HiBlackString("Username: %s \n"), color.HiBlueString(profileSelected.Username))
-	fmt.Printf(color.HiBlackString("Website: %s \n"), color.HiBlueString(profileSelected.Website))
-	fmt.Printf(color.HiBlackString("Internal ID: %s \n"), color.HiBlueString(profileSelected.Id))
+	fmt.Fprintf(color.Output, color.HiBlackString("Profile selected: %s \n"), color.HiBlueString(profileSelected.Alias))
+	fmt.Fprintf(color.Output, color.HiBlackString("Email: %s \n"), color.HiBlueString(profileSelected.Email))
+	fmt.Fprintf(color.Output, color.HiBlackString("Username: %s \n"), color.HiBlueString(profileSelected.Username))
+	fmt.Fprintf(color.Output, color.HiBlackString("Website: %s \n"), color.HiBlueString(profileSelected.Website))
+	fmt.Fprintf(color.Output, color.HiBlackString("Internal ID: %s \n"), color.HiBlueString(profileSelected.Id))
 
 }
 
@@ -290,9 +290,9 @@ func ProfilesList(cmd *cobra.Command, args []string) {
 		return
 	} else {
 
-		fmt.Println(color.HiBlackString("ID | Alias | Username | Website | Email"))
+		fmt.Fprintln(color.Output, color.HiBlackString("ID | Alias | Username | Website | Email"))
 		for key, val := range *profiles {
-			fmt.Printf(color.HiBlackString("%d. %s | %s | %s | %s | %s \n"), key, val.Id, color.HiBlueString(val.Alias), color.HiBlueString(val.Username), color.HiBlueString(val.Website), color.HiBlueString(val.Email))
+			fmt.Fprintf(color.Output, color.HiBlackString("%d. %s | %s | %s | %s | %s \n"), key, val.Id, color.HiBlueString(val.Alias), color.HiBlueString(val.Username), color.HiBlueString(val.Website), color.HiBlueString(val.Email))
 		}
 	}
 }

src/controller/revert.go

@@ -92,7 +92,7 @@ func Revert(cmd *cobra.Command, args []string) {
 
 	// Prompt the user to choose a commit
 	commit := chooseCommit(commits)
-	fmt.Printf("I will revert the commit to %s created by %s on %s \n\n", color.HiCyanString(getTitleFromCommit(commit.Message)), color.HiCyanString(commit.Author.Name), commit.Author.When.Format("Mon Jan 2 15:04:05 2006"))
+	fmt.Fprintf(color.Output, "I will revert the commit to %s created by %s on %s \n\n", color.HiCyanString(getTitleFromCommit(commit.Message)), color.HiCyanString(commit.Author.Name), commit.Author.When.Format("Mon Jan 2 15:04:05 2006"))
 
 	err = executor.GitRevert(commit.Hash.String())
 	if err != nil {

src/controller/status.go

@@ -35,7 +35,7 @@ func Status(cmd *cobra.Command, args []string) {
 		if err != nil {
 			exitOnError("Sorry, I can't get the current branch 😢", err)
 		}
-		fmt.Printf("On branch %s\n", color.HiGreenString(branch))
+		fmt.Fprintf(color.Output, "On branch %s\n", color.HiGreenString(branch))
 	}
 
 	spinner := spinner.New(spinner.CharSets[9], 100*time.Millisecond)

src/controller/whereami.go

@@ -37,10 +37,10 @@ func WhereAmI(cmd *cobra.Command, args []string) {
 		exitOnError("Sorry, I can't check if the HEAD is detached 😢", err)
 	}
 	if detached {
-		fmt.Printf("HEAD is detached (lookout mode) at %s\n", color.HiGreenString(hash))
+		fmt.Fprintf(color.Output, "HEAD is detached (lookout mode) at %s\n", color.HiGreenString(hash))
 		return
 	}
 
-	fmt.Printf("HEAD is at %s on branch %s\n", color.HiGreenString(hash), color.HiGreenString(branch))
+	fmt.Fprintf(color.Output, "HEAD is at %s on branch %s\n", color.HiGreenString(hash), color.HiGreenString(branch))
 
 }

src/prompt/prompt.go

@@ -7,13 +7,14 @@ import (
 	"runtime"
 	"strings"
 
+	"github.com/fatih/color"
 	"github.com/julien040/gut/src/print"
 	"github.com/manifoldco/promptui"
 )
 
 // InputLine prompts the user for an input and returns it
 func InputLine(message string) (string, error) {
-	fmt.Printf("%s ", message)
+	fmt.Fprintf(color.Output, "%s ", message)
 	reader := bufio.NewReader(os.Stdin)
 	input, err := reader.ReadString('\n')
 	if err != nil {