mirror of
https://github.com/flutter/packages.git
synced 2026-03-13 10:40:37 +08:00
cc3dca67f2
Makes the analysis options changes described in
https://github.com/flutter/flutter/issues/178827:
- Adding
[omit_obvious_local_variable_types](https://dart.dev/tools/linter-rules/omit_obvious_local_variable_types)
- Adding
[specify_nonobvious_local_variable_types](https://dart.dev/tools/linter-rules/specify_nonobvious_local_variable_types)
- Adding
[specify_nonobvious_property_types](https://dart.dev/tools/linter-rules/specify_nonobvious_property_types)
- Adding
[type_annotate_public_apis](https://dart.dev/tools/linter-rules/type_annotate_public_apis)
- Removing
[always_specify_types](https://dart.dev/tools/linter-rules/always_specify_types)
After those changes, makes the following repo-wide changes:
- `dart fix --apply` in all packages and in script/tool/
- `dart format` in all packages and in script/tool/
- `update-excerpts` repo tooling command to update excerpts based on the
changes to their sources
Also updates the min Flutter/Dart SDK version to 3.35/3.9 for the
following packages, to avoid `analyze` failures in the `N-2` legacy
analysis run due to what appears to be a 3.9 change in what the Dart
analyzer continues to be an obvious local type in loop iterations:
- go_router
- google_fonts
- google_identity_services_web
- google_maps_flutter_web
- local_auth_platform_interface
- metrics_center
- multicast_dns
- pigeon
- rfw
- shared_preferences
- two_dimensional_scrollables
- vector_graphics_compiler
- mustache_template
- path_parsing
Because this is causing a significant amount of format churn already, I
took this opportunity to update the repository tooling to a min Dart SDK
of 3.8 (the N-2 stable version, so the earliest version we need the
tooling to support) to pick up the new format style, so the amount of
automated formatter change is higher in script/tool/ than in the
packages.
This does contain two manual changes (other than the repo tooling min
version):
-
d700b45c7d
changes `dynamic` to `Object?` in a few places where `dynamic` caused
analyzer warnings under the new rule set.
- Updates the repo tooling to ignore `.dart_tool/` when looking for
unexpected local `analysis_options.yaml` files, to fix issues running
the repo tool's `analyze` command locally based on recent changes in
`dart` behavior.
This does not include any CHANGELOG or version updates; even though we
normally version any changes to production code, mass automated changes
like this aren't worth the churn of releasing. This includes changes to
lib/example/main.dart and to README.md excerpts; while the style changes
will be user-visible on pub.dev, it's fine for those changes to wait for
the next release of each package.
Part of https://github.com/flutter/flutter/issues/178827
35 lines
656 B
YAML
35 lines
656 B
YAML
name: flutter_plugin_tools
|
|
description: Productivity and CI utils for flutter/packages
|
|
repository: https://github.com/flutter/packages/tree/main/script/tool
|
|
version: 1.0.0
|
|
publish_to: none
|
|
|
|
dependencies:
|
|
args: ^2.1.0
|
|
async: ^2.10.0
|
|
collection: ^1.17.0
|
|
colorize: ^3.0.0
|
|
file: ^7.0.1
|
|
git: ^2.0.0
|
|
http: ^1.0.0
|
|
http_multi_server: ^3.0.1
|
|
meta: ^1.10.0
|
|
path: ^1.8.3
|
|
platform: ^3.0.2
|
|
pub_semver: ^2.0.0
|
|
pubspec_parse: ^1.4.0
|
|
quiver: ^3.0.1
|
|
test: ^1.24.0
|
|
uuid: ^4.5.1
|
|
yaml: ^3.1.0
|
|
yaml_edit: ^2.0.2
|
|
|
|
dev_dependencies:
|
|
build_runner: ^2.2.1
|
|
fake_async: ^1.3.1
|
|
matcher: ^0.12.15
|
|
mockito: ^5.4.4
|
|
|
|
environment:
|
|
sdk: ^3.8.0
|