Merge pull request #8 from craftamap/gh-7-remove-debug-logs

gh-7 only print debug logs if debug flag is set

cmd/commands/pr/create/create.go

@@ -440,7 +440,7 @@ func manageReviewers(bbrepo *bbgit.BitbucketRepo, c *client.Client, currentUser
 				nameToUUID[name] = rev
 			}
 			if len(listOfNames) == 0 {
-				logging.Warning("No reviwers to add available")
+				logging.Warning("No reviewers to add available")
 				continue
 			}
 			var addedReviewers []string

cmd/root.go

@@ -85,6 +85,7 @@ func init() {
 	rootCmd.PersistentFlags().StringVar(&username, "username", "", "username")
 	rootCmd.PersistentFlags().StringVar(&password, "password", "", "app password")
 	rootCmd.PersistentFlags().StringVar(&remoteName, "remote", "origin", "if you are in a repository and don't want to interact with the default origin, you can change it")
+	rootCmd.PersistentFlags().BoolVar(&logging.PrintDebugLogs, "debug", false, "enabling this flag allows debug logs to be printed")
 
 	err := viper.BindPFlag("username", rootCmd.PersistentFlags().Lookup("username"))
 	if err != nil {

util/logging/logging.go

@@ -6,12 +6,20 @@ import (
 	"github.com/logrusorgru/aurora"
 )
 
+var (
+	PrintDebugLogs = false
+)
+
 func Debug(message ...interface{}) {
-	fmt.Printf("%s%s\n", aurora.BgCyan(":: "+aurora.Bold("DEBUG: ").String()), fmt.Sprint(message...))
+	if PrintDebugLogs {
+		fmt.Printf("%s%s\n", aurora.BgCyan(":: "+aurora.Bold("DEBUG: ").String()), fmt.Sprint(message...))
+	}
 }
 
 func Debugf(message ...interface{}) {
-	fmt.Printf("%s%s\n", aurora.BgCyan(":: "+aurora.Bold("DEBUG: ").String()), fmt.Sprintf(message[0].(string), message[1:]...))
+	if PrintDebugLogs {
+		fmt.Printf("%s%s\n", aurora.BgCyan(":: "+aurora.Bold("DEBUG: ").String()), fmt.Sprintf(message[0].(string), message[1:]...))
+	}
 }
 
 func Error(message ...interface{}) {