Move to pkg:dart_flutter_team_lints, require Dart 2.19 (#883)

Author: kevmoo

.github/workflows/cronet.yml

@@ -36,7 +36,7 @@ jobs:
         name: Install dependencies
         run: flutter pub get
       - name: Check formatting
-        run: flutter format --output=none --set-exit-if-changed .
+        run: dart format --output=none --set-exit-if-changed .
         if: always() && steps.install.outcome == 'success'
       - name: Analyze code
         run: flutter analyze --fatal-infos

.github/workflows/dart.yml

@@ -1,4 +1,4 @@
-include: package:lints/recommended.yaml
+include: package:dart_flutter_team_lints/analysis_options.yaml
 
 analyzer:
   language:
@@ -9,36 +9,20 @@ analyzer:
 linter:
   rules:
   - avoid_bool_literals_in_conditional_expressions
-  - avoid_catching_errors
   - avoid_classes_with_only_static_members
-  - avoid_dynamic_calls
   - avoid_private_typedef_functions
   - avoid_returning_this
   - avoid_unused_constructor_parameters
   - cascade_invocations
   - comment_references
-  - directives_ordering
   - join_return_with_assignment
-  - lines_longer_than_80_chars
   - missing_whitespace_between_adjacent_strings
   - no_adjacent_strings_in_list
   - no_runtimeType_toString
-  - omit_local_variable_types
-  - only_throw_errors
-  - prefer_asserts_in_initializer_lists
   - prefer_const_constructors
   - prefer_const_declarations
   - prefer_expression_function_bodies
   - prefer_relative_imports
-  - prefer_single_quotes
-  - sort_pub_dependencies
   - test_types_in_equals
-  - throw_in_finally
-  - type_annotate_public_apis
-  - unawaited_futures
-  - unnecessary_lambdas
-  - unnecessary_parenthesis
-  - unnecessary_statements
-  - use_is_even_rather_than_modulo
   - use_string_buffers
   - use_super_parameters

analysis_options.yaml

@@ -1,3 +1,7 @@
+## 0.2.1-dev
+
+* Require Dart 2.19
+
 ## 0.2.0
 
 * Restructure `package:cronet_http` to offer a

pkgs/cronet_http/CHANGELOG.md

@@ -3,6 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 /// Tests various [CronetEngine] configurations.
+library;
 
 import 'dart:io';
 

pkgs/cronet_http/example/integration_test/cronet_configuration_test.dart

@@ -4,7 +4,7 @@ description: Demonstrates how to use the cronet_http plugin.
 publish_to: 'none'
 
 environment:
-  sdk: ">=2.17.5 <3.0.0"
+  sdk: ">=2.19.0 <3.0.0"
 
 dependencies:
   cached_network_image: ^3.2.3
@@ -16,13 +16,14 @@ dependencies:
   http: ^0.13.5
 
 dev_dependencies:
-  flutter_lints: ^1.0.0
+  dart_flutter_team_lints: ^1.0.0
   flutter_test:
     sdk: flutter
   http_client_conformance_tests:
     path: ../../http_client_conformance_tests/
   integration_test:
     sdk: flutter
+  test: ^1.23.1
 
 flutter:
   uses-material-design: true

pkgs/cronet_http/example/pubspec.yaml

@@ -50,6 +50,7 @@
 /// After the above setup, calling [Client] methods or any of the
 /// `package:http` convenient functions (e.g. [get]) will result in
 /// [CronetClient] being used on Android.
+library;
 
 import 'package:http/http.dart';
 

pkgs/cronet_http/lib/cronet_http.dart

@@ -11,6 +11,7 @@
 /// [`WidgetsFlutterBinding.ensureInitialized`](https://api.flutter.dev/flutter/widgets/WidgetsFlutterBinding/ensureInitialized.html)
 /// or
 /// [`runApp`](https://api.flutter.dev/flutter/widgets/runApp.html).
+library;
 
 import 'dart:async';
 
@@ -19,7 +20,7 @@ import 'package:http/http.dart';
 
 import 'messages.dart' as messages;
 
-late final _api = messages.HttpApi();
+final _api = messages.HttpApi();
 
 final Finalizer<String> _cronetEngineFinalizer = Finalizer(_api.freeEngine);
 

pkgs/cronet_http/lib/src/cronet_client.dart

@@ -1,11 +1,11 @@
 name: cronet_http
 description: >
   An Android Flutter plugin that provides access to the Cronet HTTP client.
-version: 0.2.0
+version: 0.2.1-dev
 repository: https://github.com/dart-lang/http/tree/master/pkgs/cronet_http
 
 environment:
-  sdk: ">=2.17.5 <3.0.0"
+  sdk: ">=2.19.0 <3.0.0"
   flutter: ">=3.0.0"
 
 dependencies:
@@ -14,7 +14,7 @@ dependencies:
   http: ^0.13.4
 
 dev_dependencies:
-  lints: ^1.0.0
+  dart_flutter_team_lints: ^1.0.0
   pigeon: ^3.2.3
   xml: ^6.1.0
   yaml_edit: ^2.0.3

pkgs/cronet_http/lib/src/messages.dart

@@ -18,6 +18,7 @@
 /// can be run to update package:cronet_http_embedded.
 ///
 /// NOTE: This script modifies the above files in place.
+library;
 
 import 'dart:io';
 

pkgs/cronet_http/pubspec.yaml

@@ -1,3 +1,7 @@
+## 0.1.2-dev
+
+* Require Dart 2.19
+
 ## 0.1.1
 
 * Add a `URLSession.sessionDescription` field.

pkgs/cronet_http/tool/prepare_for_embedded.dart

@@ -3,6 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 @Skip('Error tests cannot currently be written. See comments in this file.')
+library;
 
 import 'package:integration_test/integration_test.dart';
 import 'package:test/test.dart';

pkgs/cupertino_http/CHANGELOG.md

@@ -6,7 +6,7 @@ publish_to: 'none'
 version: 1.0.0+1
 
 environment:
-  sdk: ">=2.17.3 <3.0.0"
+  sdk: ">=2.19.0 <3.0.0"
 
 dependencies:
   cached_network_image: ^3.2.3
@@ -18,7 +18,7 @@ dependencies:
   http: ^0.13.5
 
 dev_dependencies:
-  flutter_lints: ^2.0.0
+  dart_flutter_team_lints: ^1.0.0
   flutter_test:
     sdk: flutter
   http_client_conformance_tests:

pkgs/cupertino_http/example/integration_test/error_test.dart

@@ -74,6 +74,8 @@
 ///   task.resume();
 /// }
 /// ```
+library;
+
 import 'package:http/http.dart';
 
 import 'src/cupertino_client.dart';