Tweak isColorEnabled

mislav · mislav · commit 55be0d2a9ca7 · 2020-06-22T19:18:18.000+02:00
diff --git a/utils/color.go b/utils/color.go
@@ -10,8 +10,8 @@ import (
 )
 
 var (
-	_isColorEnabled                                    bool = true
-	_isStdoutTerminal, checkedTerminal, checkedNoColor bool
+	_isColorEnabled                    bool = true
+	_isStdoutTerminal, checkedTerminal bool
 
 	// Outputs ANSI color if stdout is a tty
 	Magenta = makeColorFunc("magenta")
@@ -45,21 +45,17 @@ func NewColorable(f *os.File) io.Writer {
 func makeColorFunc(color string) func(string) string {
 	cf := ansi.ColorFunc(color)
 	return func(arg string) string {
-		if isColorEnabled() && isStdoutTerminal() {
+		if isColorEnabled() {
 			return cf(arg)
 		}
 		return arg
 	}
 }
 
 func isColorEnabled() bool {
-	if !isStdoutTerminal() {
+	if os.Getenv("NO_COLOR") != "" {
 		return false
 	}
 
-	if !checkedNoColor {
-		_isColorEnabled = os.Getenv("NO_COLOR") == ""
-		checkedNoColor = true
-	}
-	return _isColorEnabled
+	return isStdoutTerminal()
 }

Original file line number	Diff line number	Diff line change
`@@ -10,8 +10,8 @@ import (`
`10`	`10`	`)`
`11`	`11`
`12`	`12`	`var (`
`13`		`- _isColorEnabled bool = true`
`14`		`- _isStdoutTerminal, checkedTerminal, checkedNoColor bool`
	`13`	`+ _isColorEnabled bool = true`
	`14`	`+ _isStdoutTerminal, checkedTerminal bool`
`15`	`15`
`16`	`16`	`// Outputs ANSI color if stdout is a tty`
`17`	`17`	`Magenta = makeColorFunc("magenta")`
`@@ -45,21 +45,17 @@ func NewColorable(f *os.File) io.Writer {`
`45`	`45`	`func makeColorFunc(color string) func(string) string {`
`46`	`46`	`cf := ansi.ColorFunc(color)`
`47`	`47`	`return func(arg string) string {`
`48`		`- if isColorEnabled() && isStdoutTerminal() {`
	`48`	`+ if isColorEnabled() {`
`49`	`49`	`return cf(arg)`
`50`	`50`	`}`
`51`	`51`	`return arg`
`52`	`52`	`}`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`func isColorEnabled() bool {`
`56`		`- if !isStdoutTerminal() {`
	`56`	`+ if os.Getenv("NO_COLOR") != "" {`
`57`	`57`	`return false`
`58`	`58`	`}`
`59`	`59`
`60`		`- if !checkedNoColor {`
`61`		`- _isColorEnabled = os.Getenv("NO_COLOR") == ""`
`62`		`- checkedNoColor = true`
`63`		`- }`
`64`		`- return _isColorEnabled`
	`60`	`+ return isStdoutTerminal()`
`65`	`61`	`}`