refactor: use separate ci package for CI environment detection

invertase · Mar 30, 2022 · d992143 · d992143
1 parent a88ade1
commit d992143
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/packages/flutterfire_cli/lib/src/common/utils.dart b/packages/flutterfire_cli/lib/src/common/utils.dart
@@ -17,6 +17,7 @@
 
 import 'dart:io';
 import 'package:ansi_styles/ansi_styles.dart';
+import 'package:ci/ci.dart' as ci;
 import 'package:interact/interact.dart' as interact;
 import 'package:path/path.dart' show relative, normalize, windows, joinAll;
 import 'platform.dart';
@@ -48,11 +49,7 @@ extension Let<T> on T? {
 }
 
 bool get isCI {
-  final keys = currentPlatform.environment.keys;
-  return keys.contains('CI') ||
-      keys.contains('CONTINUOUS_INTEGRATION') ||
-      keys.contains('BUILD_NUMBER') ||
-      keys.contains('RUN_ID');
+  return ci.isCI;
 }
 
 int get terminalWidth {

diff --git a/packages/flutterfire_cli/pubspec.yaml b/packages/flutterfire_cli/pubspec.yaml
@@ -9,6 +9,7 @@ environment:
 dependencies:
   ansi_styles: ^0.3.0
   args: ^2.3.0
+  ci: ^0.1.0
   cli_util: ^0.3.5
   deep_pick: ^0.10.0
   file: ^6.1.2