fix: bypass isCI checks with 'CI' env set to false (#101)

Author: paescuj

index.js

@@ -54,7 +54,8 @@ vendors.forEach(function (vendor) {
 })
 
 exports.isCI = !!(
-  env.BUILD_ID || // Jenkins, Cloudbees
+  env.CI !== 'false' && // Bypass all checks if CI env is explicitly set to 'false'
+  (env.BUILD_ID || // Jenkins, Cloudbees
   env.BUILD_NUMBER || // Jenkins, TeamCity
   env.CI || // Travis CI, CircleCI, Cirrus CI, Gitlab CI, Appveyor, CodeShip, dsari
   env.CI_APP_ID || // Appflow
@@ -64,7 +65,7 @@ exports.isCI = !!(
   env.CONTINUOUS_INTEGRATION || // Travis CI, Cirrus CI
   env.RUN_ID || // TaskCluster, dsari
   exports.name ||
-  false
+  false)
 )
 
 function checkEnv (obj) {

test.js

@@ -41,6 +41,17 @@ test('Not CI', function (t) {
   t.end()
 })
 
+test('Bypass isCI checks with CI set to `false`', function (t) {
+  process.env.CI = 'false'
+
+  clearModule('./')
+  const ci = require('./')
+
+  t.equal(ci.isCI, false)
+
+  t.end()
+})
+
 test('Unknown CI', function (t) {
   process.env.CI = 'true'