Skip to content

Commit 0fff514

Browse files
authored
Enable pubspec dependency sorting lint (flutter#3983)
1 parent 737110c commit 0fff514

File tree

7 files changed

+18
-17
lines changed

7 files changed

+18
-17
lines changed

analysis_options.yaml

+3-1
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,6 @@ linter:
198198
- slash_for_doc_comments
199199
# - sort_child_properties_last # not yet tested
200200
- sort_constructors_first
201-
# - sort_pub_dependencies # prevents separating pinned transitive dependencies
202201
- sort_unnamed_constructors_first
203202
- test_types_in_equals
204203
- throw_in_finally
@@ -243,3 +242,6 @@ linter:
243242
# separately when moving to a shared file.
244243
- no_runtimeType_toString # use objectRuntimeType from package:foundation
245244
- public_member_api_docs # see https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#documentation-dartdocs-javadocs-etc
245+
# Flutter has a specific use case for dependencies that are intentionally
246+
# not sorted, which doesn't apply to this repo.
247+
- sort_pub_dependencies

packages/espresso/example/pubspec.yaml

+5-6
Original file line numberDiff line numberDiff line change
@@ -11,19 +11,18 @@ dependencies:
1111
sdk: flutter
1212

1313
dev_dependencies:
14-
flutter_test:
15-
sdk: flutter
16-
flutter_driver:
17-
sdk: flutter
18-
pedantic: ^1.10.0
19-
2014
espresso:
2115
# When depending on this package from a real application you should use:
2216
# espresso: ^x.y.z
2317
# See https://dart.dev/tools/pub/dependencies#version-constraints
2418
# The example app is bundled with the plugin so we use a path dependency on
2519
# the parent directory to use the current plugin's version.
2620
path: ../
21+
flutter_driver:
22+
sdk: flutter
23+
flutter_test:
24+
sdk: flutter
25+
pedantic: ^1.10.0
2726

2827
flutter:
2928
uses-material-design: true

packages/path_provider/path_provider/example/pubspec.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ dependencies:
1414
path: ../
1515

1616
dev_dependencies:
17-
integration_test:
18-
sdk: flutter
1917
flutter_driver:
2018
sdk: flutter
19+
integration_test:
20+
sdk: flutter
2121
pedantic: ^1.10.0
2222

2323
flutter:

packages/path_provider/path_provider/pubspec.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ dependencies:
3232
path_provider_windows: ^2.0.0
3333

3434
dev_dependencies:
35-
flutter_test:
36-
sdk: flutter
3735
flutter_driver:
3836
sdk: flutter
37+
flutter_test:
38+
sdk: flutter
3939
integration_test:
4040
sdk: flutter
4141
pedantic: ^1.10.0

packages/path_provider/path_provider_linux/example/pubspec.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ dependencies:
1919
path: ../
2020

2121
dev_dependencies:
22-
flutter_test:
23-
sdk: flutter
2422
flutter_driver:
2523
sdk: flutter
24+
flutter_test:
25+
sdk: flutter
2626
integration_test:
2727
sdk: flutter
2828

packages/path_provider/path_provider_macos/example/pubspec.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@ dependencies:
1515
path_provider_platform_interface: ^2.0.0
1616

1717
dev_dependencies:
18-
integration_test:
19-
sdk: flutter
2018
flutter_driver:
2119
sdk: flutter
20+
integration_test:
21+
sdk: flutter
2222
pedantic: ^1.8.0
2323

2424
flutter:

packages/path_provider/path_provider_windows/example/pubspec.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ dependencies:
1414
path: ../
1515

1616
dev_dependencies:
17-
integration_test:
18-
sdk: flutter
1917
flutter_driver:
2018
sdk: flutter
19+
integration_test:
20+
sdk: flutter
2121
pedantic: ^1.10.0
2222

2323
flutter:

0 commit comments

Comments
 (0)