Fix CLI warnings

brefphp · Feb 23, 2023 · 2a8b112 · 2a8b112
1 parent cf01e31
commit 2a8b112
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bref b/bref
@@ -61,7 +61,10 @@ function telemetry() {
     }
 
     // Check if we are running in CI
-    $ci = $_SERVER['CI'] || $_SERVER['CONTINUOUS_INTEGRATION'] || $_SERVER['BUILD_NUMBER'] || $_SERVER['CI_APP_ID'] || $_SERVER['CI_NAME'] || $_SERVER['RUN_ID'] || $_SERVER['BUILD_ID'];
+    $ciVars = ['CI', 'CONTINUOUS_INTEGRATION', 'BUILD_NUMBER', 'CI_APP_ID', 'CI_NAME', 'RUN_ID', 'BUILD_ID'];
+    $ci = array_reduce($ciVars, function ($carry, $item) {
+        return $carry || (isset($_SERVER[$item]) && $_SERVER[$item]);
+    }, false);
 
     $message = json_encode([
         'cli' => 'vendor/bin/bref',