fix: don't read envs when explicitly not in CI (#140)

karl-run · web-flow · commit 3fae1ac492f5 · 2025-10-02T16:12:40.000+05:30
diff --git a/index.js b/index.js
@@ -15,22 +15,24 @@ exports.name = null
 exports.isPR = null
 exports.id = null
 
-vendors.forEach(function (vendor) {
-  const envs = Array.isArray(vendor.env) ? vendor.env : [vendor.env]
-  const isCI = envs.every(function (obj) {
-    return checkEnv(obj)
-  })
+if (env.CI !== 'false') {
+  vendors.forEach(function (vendor) {
+    const envs = Array.isArray(vendor.env) ? vendor.env : [vendor.env]
+    const isCI = envs.every(function (obj) {
+      return checkEnv(obj)
+    })
 
-  exports[vendor.constant] = isCI
+    exports[vendor.constant] = isCI
 
-  if (!isCI) {
-    return
-  }
+    if (!isCI) {
+      return
+    }
 
-  exports.name = vendor.name
-  exports.isPR = checkPR(vendor)
-  exports.id = vendor.constant
-})
+    exports.name = vendor.name
+    exports.isPR = checkPR(vendor)
+    exports.id = vendor.constant
+  })
+}
 
 exports.isCI = !!(
   env.CI !== 'false' && // Bypass all checks if CI env is explicitly set to 'false'
