Merge pull request #74 from firebase/feat/pub-analyzer

HYACCCINT · web-flow · commit b701db447469 · 2024-01-19T16:31:17.000-05:00
feat: add pub analyzer action
diff --git a/.github/workflows/analyzer.yml b/.github/workflows/analyzer.yml
@@ -0,0 +1,20 @@
+name: Dart Analyzer
+
+on:
+  pull_request:
+
+jobs:
+  analyze:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: dart-lang/setup-dart@v1
+      - uses: subosito/flutter-action@v2
+        with:
+          channel: stable
+      - run: dart pub get
+      - run: dart pub global activate melos
+      - run: melos bootstrap
+      - uses: invertase/github-action-dart-analyzer@v1
+        with:
+          fatal-infos: true
diff --git a/.gitignore b/.gitignore
@@ -49,11 +49,6 @@ analytics/ios/firebase_app_id_file.json
 analytics/ios/Runner/GoogleService-Info.plist
 analytics/macos/firebase_app_id_file.json
 analytics/macos/Runner/GoogleService-Info.plist
-analytics/lib/firebase_options.dart
-dynamic_links/lib/firebase_options.dart
-
-# ignore firebase_options.dart for all examples
-**/firebase_options.dart
 
 # ignore .firebaserc for all examples
 **/.firebaserc
diff --git a/admob/test/widget_test.dart b/admob/test/widget_test.dart
@@ -13,7 +13,7 @@ import 'package:admob_quickstart/main.dart';
 void main() {
   testWidgets('Counter increments smoke test', (WidgetTester tester) async {
     // Build our app and trigger a frame.
-    await tester.pumpWidget(const MyApp());
+    await tester.pumpWidget(MyApp(state: ApplicationState()));
 
     // Verify that our counter starts at 0.
     expect(find.text('0'), findsOneWidget);
diff --git a/analytics/lib/firebase_options.dart b/analytics/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/authentication/lib/firebase_options.dart b/authentication/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/cloud_functions/lib/firebase_options.dart b/cloud_functions/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/cloud_functions/test/widget_test.dart b/cloud_functions/test/widget_test.dart
@@ -13,7 +13,7 @@ import 'package:cloud_functions_example/main.dart';
 void main() {
   testWidgets('Counter increments smoke test', (WidgetTester tester) async {
     // Build our app and trigger a frame.
-    await tester.pumpWidget(const MyApp());
+    await tester.pumpWidget(MyApp(state: ApplicationState()));
 
     // Verify that our counter starts at 0.
     expect(find.text('0'), findsOneWidget);
diff --git a/crashlytics/lib/firebase_options.dart b/crashlytics/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/dynamic_links/lib/firebase_options.dart b/dynamic_links/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/dynamic_links/lib/main.dart b/dynamic_links/lib/main.dart
@@ -32,7 +32,7 @@ const title = 'Firebase Dynamic Links Quickstart';
 
 class MyApp extends StatelessWidget {
   const MyApp({super.key, required this.state});
-  
+
   final ApplicationState state;
 
   // This widget is the root of your application.
@@ -60,7 +60,7 @@ class MyApp extends StatelessWidget {
 
 class MyHomePage extends StatelessWidget {
   const MyHomePage({super.key, required this.title, required this.state});
-  
+
   // This widget is the home page of your application. It is stateful, meaning
   // that it has a State object (defined below) that contains fields that affect
   // how it looks.
diff --git a/firestore/lib/firebase_options.dart b/firestore/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/inappmessaging/lib/firebase_options.dart b/inappmessaging/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/melos.yaml b/melos.yaml
@@ -1,15 +1,24 @@
-name: quickstart-flutter
+name: quickstart_flutter
 repository: https://github.com/firebase/quickstart-flutter
 
 packages:
-- authentication/**
-- crashlytics/**
-- firestore/**
-- remote_config/**
-- storage/**
+  - admob
+  - analytics
+  - authentication
+  - cloud_functions
+  - crashlytics
+  - dynamic_links
+  - firestore
+  - inappmessaging
+  - messaging
+  - performance
+  - remote_config
+  - storage
 
 scripts:
   analyze: melos exec -- "dart analyze ."
+  
+  format: melos exec -- "dart format ."
 
   lint:all:
     run: melos run analyze && melos run format
diff --git a/messaging/lib/firebase_options.dart b/messaging/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/performance/lib/firebase_options.dart b/performance/lib/firebase_options.dart
@@ -0,0 +1,8 @@
+import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
+
+/// Run 'flutterfire configure' to generate platform-specific FirebaseOptions.
+class DefaultFirebaseOptions {
+  static FirebaseOptions get currentPlatform {
+    throw StateError("firebase_options.dart not generated. Did you run 'flutterfire configure'?");
+  }
+}
diff --git a/performance/lib/firebase_status.dart b/performance/lib/firebase_status.dart
@@ -7,7 +7,7 @@ class ResponseJson {
     if (json['response'] != null) {
       response = <FirebaseStatus>[];
       json['response'].forEach((v) {
-        response!.add(new FirebaseStatus.fromJson(v));
+        response!.add(FirebaseStatus.fromJson(v));
       });
     }
   }
@@ -58,11 +58,11 @@ class FirebaseStatus {
     if (json['updates'] != null) {
       updates = <Updates>[];
       json['updates'].forEach((v) {
-        updates!.add(new Updates.fromJson(v));
+        updates!.add(Updates.fromJson(v));
       });
     }
     mostRecentUpdate = json['most_recent_update'] != null
-        ? new Updates.fromJson(json['most_recent_update'])
+        ? Updates.fromJson(json['most_recent_update'])
         : null;
     statusImpact = json['status_impact'];
     severity = json['severity'];
@@ -71,36 +71,36 @@ class FirebaseStatus {
     if (json['affected_products'] != null) {
       affectedProducts = <AffectedProducts>[];
       json['affected_products'].forEach((v) {
-        affectedProducts!.add(new AffectedProducts.fromJson(v));
+        affectedProducts!.add(AffectedProducts.fromJson(v));
       });
     }
     uri = json['uri'];
   }
 
   Map<String, dynamic> toJson() {
-    final Map<String, dynamic> data = new Map<String, dynamic>();
-    data['id'] = this.id;
-    data['number'] = this.number;
-    data['begin'] = this.begin;
-    data['created'] = this.created;
-    data['end'] = this.end;
-    data['modified'] = this.modified;
-    data['external_desc'] = this.externalDesc;
-    if (this.updates != null) {
-      data['updates'] = this.updates!.map((v) => v.toJson()).toList();
+    final Map<String, dynamic> data = <String, dynamic>{};
+    data['id'] = id;
+    data['number'] = number;
+    data['begin'] = begin;
+    data['created'] = created;
+    data['end'] = end;
+    data['modified'] = modified;
+    data['external_desc'] = externalDesc;
+    if (updates != null) {
+      data['updates'] = updates!.map((v) => v.toJson()).toList();
     }
-    if (this.mostRecentUpdate != null) {
-      data['most_recent_update'] = this.mostRecentUpdate!.toJson();
+    if (mostRecentUpdate != null) {
+      data['most_recent_update'] = mostRecentUpdate!.toJson();
     }
-    data['status_impact'] = this.statusImpact;
-    data['severity'] = this.severity;
-    data['service_key'] = this.serviceKey;
-    data['service_name'] = this.serviceName;
-    if (this.affectedProducts != null) {
+    data['status_impact'] = statusImpact;
+    data['severity'] = severity;
+    data['service_key'] = serviceKey;
+    data['service_name'] = serviceName;
+    if (affectedProducts != null) {
       data['affected_products'] =
-          this.affectedProducts!.map((v) => v.toJson()).toList();
+          affectedProducts!.map((v) => v.toJson()).toList();
     }
-    data['uri'] = this.uri;
+    data['uri'] = uri;
     return data;
   }
 }
@@ -123,12 +123,12 @@ class Updates {
   }
 
   Map<String, dynamic> toJson() {
-    final Map<String, dynamic> data = new Map<String, dynamic>();
-    data['created'] = this.created;
-    data['modified'] = this.modified;
-    data['when'] = this.when;
-    data['text'] = this.text;
-    data['status'] = this.status;
+    final Map<String, dynamic> data = <String, dynamic>{};
+    data['created'] = created;
+    data['modified'] = modified;
+    data['when'] = when;
+    data['text'] = text;
+    data['status'] = status;
     return data;
   }
 }
@@ -145,9 +145,9 @@ class AffectedProducts {
   }
 
   Map<String, dynamic> toJson() {
-    final Map<String, dynamic> data = new Map<String, dynamic>();
-    data['title'] = this.title;
-    data['id'] = this.id;
+    final Map<String, dynamic> data = <String, dynamic>{};
+    data['title'] = title;
+    data['id'] = id;
     return data;
   }
 }
diff --git a/pubspec.lock b/pubspec.lock
diff --git a/pubspec.yaml b/pubspec.yaml
diff --git a/remote_config/lib/firebase_options.dart b/remote_config/lib/firebase_options.dart
diff --git a/storage/lib/firebase_options.dart b/storage/lib/firebase_options.dart
