Merge pull request #32 from oliverpool/patch-1

matryer · web-flow · commit 03891c89c4c0 · 2019-11-20T14:45:54.000Z
Fix handling of the IS_NO_COLOR env variable
diff --git a/is.go b/is.go
@@ -74,8 +74,8 @@ type I struct {
 var noColorFlag bool
 
 func init() {
-	colorEnv := os.Getenv("IS_NO_COLOR") != "false"
-	flag.BoolVar(&noColorFlag, "nocolor", colorEnv, "turns off colors")
+	envNoColor := os.Getenv("IS_NO_COLOR") == "true"
+	flag.BoolVar(&noColorFlag, "nocolor", envNoColor, "turns off colors")
 }
 
 // New makes a new testing helper using the specified

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,8 @@ type I struct {`
`74`	`74`	`var noColorFlag bool`
`75`	`75`
`76`	`76`	`func init() {`
`77`		`- colorEnv := os.Getenv("IS_NO_COLOR") != "false"`
`78`		`- flag.BoolVar(&noColorFlag, "nocolor", colorEnv, "turns off colors")`
	`77`	`+ envNoColor := os.Getenv("IS_NO_COLOR") == "true"`
	`78`	`+ flag.BoolVar(&noColorFlag, "nocolor", envNoColor, "turns off colors")`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`// New makes a new testing helper using the specified`