remove global verbose flag

It is not widely used. Just in one command.
It is to us to make decision what to show to user.
If really needed we should put that flag only to the specific command.

Author: ianic

cli/cmd/root.go

@@ -39,14 +39,10 @@ func Execute(ctx context.Context, version string) error {
 	}
 
 	// register global flags
-	var verbose, noColor bool
-	cmd.PersistentFlags().BoolVarP(&verbose, "verbose", "v", false, "verbose log output")
+	var noColor bool
 	cmd.PersistentFlags().BoolVar(&noColor, "no-color", false, "don't use color in log output")
 	// use global flags
 	cobra.OnInitialize(func() {
-		if verbose {
-			log.EnableDebugLogLevel()
-		}
 		if noColor {
 			log.UI.DisableColor()
 		}

cli/log/log.go

@@ -12,14 +12,6 @@ import (
 	"github.com/pkg/errors"
 )
 
-var (
-	debugLogLevelEnabled = false
-)
-
-func EnableDebugLogLevel() {
-	debugLogLevelEnabled = true
-}
-
 func (u *UILogger) DisableColor() {
 	print := func(a ...interface{}) {
 		fmt.Print(a...)
@@ -110,9 +102,6 @@ func (u *UILogger) Info(format string, v ...interface{}) {
 }
 
 func (u *UILogger) Debug(format string, v ...interface{}) {
-	if !debugLogLevelEnabled {
-		return
-	}
 	u.debugLog(fmt.Sprintf(format, v...))
 }
 
@@ -149,16 +138,13 @@ func (u *UILogger) Backend(msg string) {
 		u.infoLog(fmt.Sprintf("λ %s", msg))
 		return
 	}
-	if l.Level == levelDebug && !debugLogLevelEnabled {
-		return
-	}
 	c := u.levelColor(l.Level)
 	c(fmt.Sprintf("λ %s", l.Msg))
 }
 
 const (
-	levelInfo  = "info"
 	levelDebug = "debug"
+	levelInfo  = "info"
 	levelError = "error"
 )