Merge pull request #2639 from felipecrs/fix-ci-color

Not rely on CI=true for TTY check
postmanlabs · Apr 4, 2021 · bb898b5 · bb898b5
2 parents e0f1f25 + 92ccf32
commit bb898b5
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/reporters/cli/cli-utils.js b/lib/reporters/cli/cli-utils.js
@@ -93,14 +93,13 @@ cliUtils = {
      * color:     |  noTTY:
      * 'on'      -> false
      * 'off'     -> true
-     * otherwise -> Based on CI or isTTY.
+     * otherwise -> Based on isTTY.
      *
      * @param {String} color - A flag to indicate usage of the --color option.
      * @returns {Boolean} - A boolean value depicting the result of the noTTY check.
      */
     noTTY: function (color) {
-        return (color === 'off') || (color !== 'on') &&
-            (Boolean(process.env.CI) || !process.stdout.isTTY); // eslint-disable-line no-process-env
+        return (color === 'off') || (color !== 'on') && (!process.stdout.isTTY);
     },
 
     /**