Move to dart_flutter_team_lints with associated cleanup (google#1282)

Author: kevmoo

_test_yaml/pubspec.yaml

@@ -10,9 +10,9 @@ dev_dependencies:
   build_runner: ^2.0.0
   build_verify: ^3.0.0
   checked_yaml: any
+  dart_flutter_team_lints: ^0.1.0
   json_annotation: ^4.7.0
   json_serializable: any
-  lints: ^2.0.0
   path: ^1.8.2
   test: ^1.6.0
   yaml: ^3.0.0

analysis_options.yaml

@@ -1,14 +1,12 @@
-include: package:lints/recommended.yaml
+# See https://dart.dev/guides/language/analysis-options
+include: package:dart_flutter_team_lints/analysis_options.yaml
 
 analyzer:
   language:
     strict-casts: true
 
 linter:
   rules:
-    - always_declare_return_types
-    - avoid_catching_errors
-    - avoid_dynamic_calls
     - avoid_private_typedef_functions
     - avoid_redundant_argument_values
     - avoid_returning_null
@@ -20,38 +18,24 @@ linter:
     - combinators_ordering
     - comment_references
     - dangling_library_doc_comments
-    - directives_ordering
     - implicit_call_tearoffs
     - join_return_with_assignment
     - library_annotations
-    - lines_longer_than_80_chars
     - literal_only_boolean_expressions
     - missing_whitespace_between_adjacent_strings
     - no_runtimeType_toString
-    - omit_local_variable_types
-    - only_throw_errors
     - package_api_docs
-    - prefer_asserts_in_initializer_lists
     - prefer_const_constructors
     - prefer_const_declarations
     - prefer_expression_function_bodies
     - prefer_final_locals
     - prefer_relative_imports
-    - prefer_single_quotes
     - sort_child_properties_last
-    - sort_pub_dependencies
     - test_types_in_equals
-    - throw_in_finally
-    - type_annotate_public_apis
-    - unawaited_futures
-    - unnecessary_lambdas
     - unnecessary_library_directive
-    - unnecessary_parenthesis
-    - unnecessary_statements
     - unreachable_from_main
     - unsafe_html
     - use_full_hex_values_for_flutter_colors
-    - use_is_even_rather_than_modulo
     - use_string_buffers
     - use_string_in_part_of_directives
     - use_super_parameters

checked_yaml/pubspec.yaml

@@ -16,8 +16,8 @@ dependencies:
 dev_dependencies:
   build_runner: ^2.0.0
   build_verify: ^3.0.0
+  dart_flutter_team_lints: ^0.1.0
   json_serializable: ^6.0.0
-  lints: ^2.0.0
   path: ^1.0.0
   test: ^1.16.0
   test_process: ^2.0.0

example/pubspec.yaml

@@ -18,11 +18,13 @@ dev_dependencies:
   # Used by tests. Not required to use `json_serializable`.
   build_verify: ^3.0.0
 
+  # Not required to use `json_serializable`.
+  dart_flutter_team_lints: ^0.1.0
+
   # REQUIRED!
   json_serializable: ^6.6.0
 
   # Not required to use `json_serializable`.
-  lints: ^2.0.0
   path: ^1.8.0
   test: ^1.16.0
 

json_annotation/pubspec.yaml

@@ -12,7 +12,7 @@ dependencies:
   meta: ^1.4.0
 
 dev_dependencies:
-  lints: ^2.0.0
+  dart_flutter_team_lints: ^0.1.0
 # When changing JsonSerializable class.
 #  build_runner: ^2.0.0
 #  json_serializable: any

json_serializable/pubspec.yaml

@@ -29,8 +29,8 @@ dev_dependencies:
     path: ../shared_test
   build_runner: ^2.0.0
   build_verify: ^3.0.0
+  dart_flutter_team_lints: ^0.1.0
   dart_style: ^2.0.0
-  lints: ^2.0.0
   logging: ^1.0.0
   source_gen_test: ^1.0.0
   test: ^1.16.0

shared_test/pubspec.yaml

@@ -8,4 +8,4 @@ dependencies:
   test: ^1.6.0
 
 dev_dependencies:
-  lints: ^2.0.0
+  dart_flutter_team_lints: ^0.1.0