Actually let you disable colors with jake (#17414)

weswigham · web-flow · commit 2d4938d5c00e · 2017-07-25T14:35:22.000-07:00
* Actually let you disable colors with jake * @Andy-MS revision
diff --git a/Jakefile.js b/Jakefile.js
@@ -840,7 +840,8 @@ function runConsoleTests(defaultReporter, runInParallel) {
         testTimeout = 800000;
     }
 
-    var colors = process.env.colors || process.env.color || true;
+    var colorsFlag = process.env.color || process.env.colors;
+    var colors = colorsFlag !== "false" && colorsFlag !== "0";
     var reporter = process.env.reporter || process.env.r || defaultReporter;
     var bail = process.env.bail || process.env.b;
     var lintFlag = process.env.lint !== 'false';

Original file line number	Diff line number	Diff line change
`@@ -840,7 +840,8 @@ function runConsoleTests(defaultReporter, runInParallel) {`
`840`	`840`	`testTimeout = 800000;`
`841`	`841`	`}`
`842`	`842`
`843`		`- var colors = process.env.colors \|\| process.env.color \|\| true;`
	`843`	`+ var colorsFlag = process.env.color \|\| process.env.colors;`
	`844`	`+ var colors = colorsFlag !== "false" && colorsFlag !== "0";`
`844`	`845`	`var reporter = process.env.reporter \|\| process.env.r \|\| defaultReporter;`
`845`	`846`	`var bail = process.env.bail \|\| process.env.b;`
`846`	`847`	`var lintFlag = process.env.lint !== 'false';`