Paul Berry
bd65b86ada
(Part of https://github.com/dart-lang/sdk/issues/63288) This change migrates the analyzer_cli package to use the new constructor declaration syntax, described in https://github.com/dart-lang/language/blob/main/accepted/future-releases/primary-constructors/feature-specification.md#abbreviations-of-in-body-constructor-declarations. This change was performed in an automated fashion, by (a) bumping the package's SDK constraint to `3.13.0-0`, (b) enabling the lints `unnecessary_type_name_in_constructor` and `unnecessary_const_in_enum_constructor`, (c) fixing the resulting lint failures using `dart fix`, and then (d) reformatting the affected files. To ease code review, I've reverted unrelated formatting changes. Change-Id: I563e97f19c03ad0672dba271de303b786a6a6964 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/508367 Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
32 lines
845 B
YAML
32 lines
845 B
YAML
include: package:lints/recommended.yaml
|
|
|
|
analyzer:
|
|
language:
|
|
strict-casts: true
|
|
strict-inference: true
|
|
errors:
|
|
# We import heavily from package:analyzer/src.
|
|
implementation_imports: ignore
|
|
non_constant_identifier_names: ignore
|
|
# Ignoring this for all developers means developers don't need to ignore
|
|
# TODOs in their IDE settings.
|
|
todo: ignore
|
|
exclude:
|
|
- test/data/**
|
|
|
|
linter:
|
|
rules:
|
|
- unnecessary_type_name_in_constructor
|
|
- unnecessary_const_in_enum_constructor
|
|
- always_declare_return_types
|
|
- avoid_unused_constructor_parameters
|
|
- directives_ordering
|
|
- flutter_style_todos
|
|
- omit_local_variable_types
|
|
- prefer_single_quotes
|
|
- unawaited_futures
|
|
- unnecessary_ignore
|
|
- unnecessary_library_directive
|
|
- unnecessary_parenthesis
|
|
- unreachable_from_main
|