feat: decrease Dart SDK minimum from 2.17 to 2.14 (#34)

Apps with Flutter 2.x are using Dart 2.14, like [Sharezone](https://github.com/SharezoneApp/sharezone-app). There was no real to use Dart 2.17. So we can just decrease the minimum.

Author: nilsreichardt

.github/workflows/pull_request.yaml

@@ -3,6 +3,9 @@ name: pull-request-ci
 on:
   pull_request:
 
+env:
+  FLUTTER_VERSION: 2.5.0 # Uses Dart SDK v2.14.0
+
 jobs:
   test:
     runs-on: ubuntu-latest
@@ -11,7 +14,6 @@ jobs:
       - uses: subosito/flutter-action@v1
         with:
           flutter-version: ${{ env.FLUTTER_VERSION }}
-          channel: ${{ env.FLUTTER_CHANNEL }}
 
       - name: Install Melos
         run: dart pub global activate melos
@@ -29,7 +31,6 @@ jobs:
       - uses: subosito/flutter-action@v1
         with:
           flutter-version: ${{ env.FLUTTER_VERSION }}
-          channel: ${{ env.FLUTTER_CHANNEL }}
 
       - name: Install Melos
         run: dart pub global activate melos
@@ -47,7 +48,6 @@ jobs:
       - uses: subosito/flutter-action@v1
         with:
           flutter-version: ${{ env.FLUTTER_VERSION }}
-          channel: ${{ env.FLUTTER_CHANNEL }}
 
       - name: Install Melos
         run: dart pub global activate melos

packages/app_preview_example/lib/main.dart

@@ -5,7 +5,7 @@ void main() {
 }
 
 class MyApp extends StatelessWidget {
-  const MyApp({super.key});
+  const MyApp({Key? key}) : super(key: key);
 
   @override
   Widget build(BuildContext context) {
@@ -20,7 +20,7 @@ class MyApp extends StatelessWidget {
 }
 
 class MyHomePage extends StatefulWidget {
-  const MyHomePage({super.key, required this.title});
+  const MyHomePage({Key? key, required this.title}) : super(key: key);
 
   final String title;
 

packages/app_preview_example/pubspec.lock

@@ -14,7 +14,7 @@ packages:
       name: async
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "2.8.2"
+    version: "2.8.1"
   boolean_selector:
     dependency: transitive
     description:
@@ -28,7 +28,7 @@ packages:
       name: characters
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "1.2.0"
+    version: "1.1.0"
   charcode:
     dependency: transitive
     description:
@@ -49,14 +49,14 @@ packages:
       name: collection
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "1.16.0"
+    version: "1.15.0"
   fake_async:
     dependency: transitive
     description:
       name: fake_async
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "1.3.0"
+    version: "1.2.0"
   flutter:
     dependency: "direct main"
     description: flutter
@@ -73,21 +73,14 @@ packages:
       name: lints
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "2.0.0"
+    version: "1.0.1"
   matcher:
     dependency: transitive
     description:
       name: matcher
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "0.12.11"
-  material_color_utilities:
-    dependency: transitive
-    description:
-      name: material_color_utilities
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "0.1.4"
+    version: "0.12.10"
   meta:
     dependency: transitive
     description:
@@ -101,7 +94,7 @@ packages:
       name: path
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "1.8.1"
+    version: "1.8.0"
   sky_engine:
     dependency: transitive
     description: flutter
@@ -113,7 +106,7 @@ packages:
       name: source_span
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "1.8.2"
+    version: "1.8.1"
   stack_trace:
     dependency: transitive
     description:
@@ -148,13 +141,20 @@ packages:
       name: test_api
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "0.4.9"
+    version: "0.4.2"
+  typed_data:
+    dependency: transitive
+    description:
+      name: typed_data
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "1.3.0"
   vector_math:
     dependency: transitive
     description:
       name: vector_math
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "2.1.2"
+    version: "2.1.0"
 sdks:
-  dart: ">=2.17.5 <3.0.0"
+  dart: ">=2.14.0 <3.0.0"

packages/app_preview_example/pubspec.yaml

@@ -4,7 +4,7 @@ publish_to: 'none'
 version: 1.0.0+1
 
 environment:
-  sdk: ">=2.17.5 <3.0.0"
+  sdk: ">=2.14.0 <3.0.0"
 
 dependencies:
   flutter:

packages/app_preview_lints/lib/analysis_options.yaml

@@ -8,4 +8,3 @@ linter:
     # The following lints are for a consistent code style.
     - sort_constructors_first
     - prefer_single_quotes
-    - use_super_parameters

packages/app_preview_lints/pubspec.yaml

@@ -2,7 +2,7 @@ name: app_preview_lints
 description: Set of lint rules for for the app preview CLI.
 
 environment:
-  sdk: ">=2.17.0 <3.0.0"
+  sdk: ">=2.14.0 <3.0.0"
 
 dependencies:
-  lints: ^2.0.0
+  lints: ^1.0.1

packages/codemagic_app_preview/lib/src/builds/build.dart

@@ -28,7 +28,7 @@ class Build {
   factory Build.fromJson(Map<String, dynamic> json) {
     return Build(
       url: json['url'],
-      platform: BuildPlatform.fromType(json['type']),
+      platform: getBuildPlatform(json['type']),
     );
   }
 

packages/codemagic_app_preview/lib/src/builds/build_platform.dart

@@ -8,28 +8,34 @@ enum BuildPlatform {
   /// The build is for the iOS platform.
   ///
   /// File formats are `.ipa`.
-  ios;
-
-  factory BuildPlatform.fromType(String type) {
-    switch (type) {
-      // todo: is this really aab or something else?
-      case 'aab':
-      case 'apk':
-        return android;
-      case 'ipa':
-        return ios;
-      default:
-        throw Exception('Unknown build platform type: $type');
-    }
-  }
+  ios,
+}
 
+/// Extension methods for [BuildPlatform].
+///
+/// When using Dart 2.17 as minimum version, should this be refactored to enum
+/// class, like: https://github.com/nilsreichardt/codemagic-app-preview/blob/8996c2219af619acfb0a37df83e83041202653e4/packages/codemagic_app_preview/lib/src/builds/build_platform.dart
+extension BuildPlatformExtension on BuildPlatform {
   /// Returns the name of the platform, e.g. "iOS" for [ios].
-  String get name {
+  String get platformName {
     switch (this) {
-      case android:
+      case BuildPlatform.android:
         return 'Android';
-      case ios:
+      case BuildPlatform.ios:
         return 'iOS';
     }
   }
 }
+
+/// Returns the [BuildPlatform] for the given [fileExtension].
+BuildPlatform getBuildPlatform(String fileExtension) {
+  switch (fileExtension) {
+    case 'aab':
+    case 'apk':
+      return BuildPlatform.android;
+    case 'ipa':
+      return BuildPlatform.ios;
+    default:
+      throw Exception('Unknown build platform file extension: $fileExtension');
+  }
+}

packages/codemagic_app_preview/lib/src/comment/comment_builder.dart

@@ -1,4 +1,5 @@
 import 'package:codemagic_app_preview/src/builds/build.dart';
+import 'package:codemagic_app_preview/src/builds/build_platform.dart';
 import 'package:codemagic_app_preview/src/environment_variable/environment_variable_accessor.dart';
 
 class CommentBuilder {
@@ -50,7 +51,7 @@ class CommentBuilder {
 
     table.write('|');
     for (final build in builds) {
-      table.write(' ${build.platform.name} |');
+      table.write(' ${build.platform.platformName} |');
     }
 
     table.write('\n|');

packages/codemagic_app_preview/lib/src/comment/comment_poster.dart

@@ -20,7 +20,7 @@ class CommentPoster {
     final shouldEdit = previousComment != null;
 
     if (shouldEdit) {
-      await _gitHubApi.editComment(previousComment.id, comment);
+      await _gitHubApi.editComment(previousComment!.id, comment);
     } else {
       await _gitHubApi.postComment(pullRequestId, comment);
     }