🐛 fix double error display if called outside a git repo without the global flag

Author: Shieldine

cmd/check.go

@@ -18,6 +18,7 @@ import (
 	"fmt"
 	"github.com/Shieldine/git-profile/internal"
 	"github.com/spf13/cobra"
+	"os"
 )
 
 var checkCmd = &cobra.Command{
@@ -41,6 +42,11 @@ func runCheck(cmd *cobra.Command, _ []string) {
 		email, emailErr = internal.GetGlobalUserEmail()
 		fmt.Println("Global Git Configuration:")
 	} else {
+		if !internal.CheckGitRepo() {
+			fmt.Println("error: not a git repository")
+			os.Exit(1)
+		}
+
 		name, nameErr = internal.GetUserName()
 		email, emailErr = internal.GetUserEmail()
 		fmt.Println("Local Git Configuration:")

cmd/root.go

@@ -20,7 +20,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-var version = "1.2.1"
+var version = "1.2.2"
 
 var rootCmd = &cobra.Command{
 	Use:     "git-profile",

cmd/set.go

@@ -41,10 +41,15 @@ var setCmd = &cobra.Command{
 // It sets the Git user configuration (name and email) based on the specified profile.
 // If --global flag is used, it sets the global Git configuration; otherwise, it sets the local repository configuration.
 func runSet(cmd *cobra.Command, args []string) {
-	profileName := args[0]
-
 	global, _ := cmd.Flags().GetBool("global")
 
+	if !global && !internal.CheckGitRepo() {
+		fmt.Println("error: not a git repository")
+		os.Exit(1)
+	}
+
+	profileName := args[0]
+
 	profile := internal.GetProfileByName(profileName)
 
 	if (models.ProfileConfig{}) == profile {

cmd/tempset.go

@@ -42,9 +42,15 @@ If you don't pass them, you will be asked to provide a name and an email.
 // If --global flag is used, it sets the global Git configuration; otherwise, it sets the local repository configuration.
 // Credentials can be provided via flags or will be prompted interactively.
 func runTempSet(cmd *cobra.Command, _ []string) {
-	reader := bufio.NewReader(os.Stdin)
 	global, _ := cmd.Flags().GetBool("global")
 
+	if !global && !internal.CheckGitRepo() {
+		fmt.Println("error: not a git repository")
+		os.Exit(1)
+	}
+
+	reader := bufio.NewReader(os.Stdin)
+
 	if name == "" {
 		var currentName string
 		var err error

cmd/unset.go

@@ -18,6 +18,7 @@ import (
 	"fmt"
 	"github.com/Shieldine/git-profile/internal"
 	"github.com/spf13/cobra"
+	"os"
 )
 
 var unsetCmd = &cobra.Command{
@@ -38,6 +39,11 @@ func runUnset(cmd *cobra.Command, _ []string) {
 	if global {
 		fmt.Println("warning: removing global git credentials")
 	} else {
+		if !internal.CheckGitRepo() {
+			fmt.Println("error: not a git repository")
+			os.Exit(1)
+		}
+
 		fmt.Println("warning: git will default to global credentials without local configuration")
 	}
 
@@ -50,8 +56,6 @@ func runUnset(cmd *cobra.Command, _ []string) {
 	if err != nil {
 		fmt.Println(err)
 	}
-
-	fmt.Println("Process complete.")
 }
 
 // init initializes the unset command and adds it to the root command.