Skip to content

Commit

Permalink
Mass pubspec.yaml cleanup (flutter#3927)
Browse files Browse the repository at this point in the history
Regularizes the pubspec.yaml files of the plugins:
- Changes from `homepage` to `repository` to improve the linking on pub.dev.
- Adds an `issue_tracker` link that links directly to a search on that plugin's label, to improve the linking on pub.dev.
- Standardizes the section order as `environment`, `flutter`, `dependencies`, `dev_dependencies`. This matches the current template, except that `flutter` is higher as it's critical information in a plugin (which should potentially be revisited in the template).
- Sorts dependencies within each section.
- Updates the Flutter SDK versions to 2.0.0; in practice this is already the case due to the Dart SDK version requirement, so this just makes it less confusing.
- Standardizes the location of `version` in the initial block.

Changes that were considered out of scope:
- Actually cleaning up dependencies (e.g., removing meta where it's no longer needed). The goal was to make this PR as mechanical as possible, and to not change any actual behavior).
- The example/ pubspecs (just to limit the scope of the PR; we should eventually clean those up too).
- Plugins that have Plus Plugin equivalents, per repo policy, as this is a non-critical change.

This does not update versions; these changes can be picked up the next time the plugins happen to be published.

This also doesn't add automated enforcement. We should consider that in the future though; filed as flutter/flutter#82913
  • Loading branch information
stuartmorgan authored May 21, 2021
1 parent 5c778ea commit 388f916
Show file tree
Hide file tree
Showing 46 changed files with 393 additions and 365 deletions.
36 changes: 17 additions & 19 deletions packages/camera/camera/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,41 +2,39 @@ name: camera
description: A Flutter plugin for getting information about and controlling the
camera on Android and iOS. Supports previewing the camera feed, capturing images, capturing video,
and streaming image buffers to dart.
repository: https://github.com/flutter/plugins/tree/master/packages/camera/camera
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22
version: 0.8.1+1
homepage: https://github.com/flutter/plugins/tree/master/packages/camera/camera

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.camera
pluginClass: CameraPlugin
ios:
pluginClass: CameraPlugin

dependencies:
camera_platform_interface: ^2.0.0
flutter:
sdk: flutter

camera_platform_interface: ^2.0.0

pedantic: ^1.10.0
quiver: ^3.0.0

dev_dependencies:
video_player: ^2.0.0
flutter_test:
sdk: flutter
flutter_driver:
sdk: flutter

# TODO(mvanbeusekom): Update to stable 5.0.0 release when dependency conflict
# with flutter_driver has been resolved:
# Because mockito >=5.0.0 depends on analyzer ^1.0.0 which depends on crypto ^3.0.0
# every version of flutter_driver from sdk depends on crypto 2.1.5.
mockito: ^5.0.0-nullsafety.7
plugin_platform_interface: ^2.0.0

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.camera
pluginClass: CameraPlugin
ios:
pluginClass: CameraPlugin

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.22.0"
video_player: ^2.0.0
15 changes: 8 additions & 7 deletions packages/camera/camera_platform_interface/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,24 +1,25 @@
name: camera_platform_interface
description: A common platform interface for the camera plugin.
homepage: https://github.com/flutter/plugins/tree/master/packages/camera/camera_platform_interface
repository: https://github.com/flutter/plugins/tree/master/packages/camera/camera_platform_interface
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22
# NOTE: We strongly prefer non-breaking changes, even at the expense of a
# less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
version: 2.0.1

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=2.0.0"

dependencies:
cross_file: ^0.3.1
flutter:
sdk: flutter
meta: ^1.3.0
plugin_platform_interface: ^2.0.0
cross_file: ^0.3.1
stream_transform: ^2.0.0

dev_dependencies:
async: ^2.5.0
flutter_test:
sdk: flutter
async: ^2.5.0
pedantic: ^1.10.0

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=1.22.0"
21 changes: 10 additions & 11 deletions packages/espresso/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,12 +1,19 @@
name: espresso
description: Java classes for testing Flutter apps using Espresso.
repository: https://github.com/flutter/plugins/tree/master/packages/espresso
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+espresso%22
version: 0.1.0+2
repository: https://github.com/flutter/plugins/tree/master/packages
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+espresso+label%3A%22p%3A+espresso%22

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
android:
package: com.example.espresso
pluginClass: EspressoPlugin

dependencies:
flutter:
Expand All @@ -16,11 +23,3 @@ dev_dependencies:
flutter_test:
sdk: flutter
pedantic: ^1.10.0

# The following section is specific to Flutter.
flutter:
plugin:
platforms:
android:
package: com.example.espresso
pluginClass: EspressoPlugin
19 changes: 10 additions & 9 deletions packages/file_selector/file_selector/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,27 +1,28 @@
name: file_selector
description: Flutter plugin for opening and saving files.
homepage: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector
repository: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+file_selector%22
version: 0.8.2

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
web:
default_package: file_selector_web

dependencies:
flutter:
sdk: flutter
file_selector_platform_interface: ^2.0.0
file_selector_web: ^0.8.1
flutter:
sdk: flutter

dev_dependencies:
flutter_test:
sdk: flutter
test: ^1.16.3
plugin_platform_interface: ^2.0.0
pedantic: ^1.10.0

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
plugin_platform_interface: ^2.0.0
test: ^1.16.3
Original file line number Diff line number Diff line change
@@ -1,24 +1,25 @@
name: file_selector_platform_interface
description: A common platform interface for the file_selector plugin.
homepage: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector_platform_interface
repository: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector_platform_interface
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+file_selector%22
# NOTE: We strongly prefer non-breaking changes, even at the expense of a
# less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
version: 2.0.2

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=2.0.0"

dependencies:
cross_file: ^0.3.0
flutter:
sdk: flutter
meta: ^1.3.0
http: ^0.13.0
meta: ^1.3.0
plugin_platform_interface: ^2.0.0
cross_file: ^0.3.0

dev_dependencies:
test: ^1.16.3
flutter_test:
sdk: flutter
pedantic: ^1.10.0

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
test: ^1.16.3
11 changes: 6 additions & 5 deletions packages/file_selector/file_selector_web/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
name: file_selector_web
description: Web platform implementation of file_selector
homepage: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector_web
repository: https://github.com/flutter/plugins/tree/master/packages/file_selector/file_selector_web
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+file_selector%22
version: 0.8.1

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
Expand All @@ -22,7 +27,3 @@ dev_dependencies:
flutter_test:
sdk: flutter
pedantic: ^1.10.0

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
19 changes: 10 additions & 9 deletions packages/flutter_plugin_android_lifecycle/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,11 +1,19 @@
name: flutter_plugin_android_lifecycle
description: Flutter plugin for accessing an Android Lifecycle within other plugins.
repository: https://github.com/flutter/plugins/tree/master/packages/flutter_plugin_android_lifecycle
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+flutter_plugin_android_lifecycle%22
version: 2.0.2
homepage: https://github.com/flutter/plugins/tree/master/packages/flutter_plugin_android_lifecycle

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.flutter_plugin_android_lifecycle
pluginClass: FlutterAndroidLifecyclePlugin

dependencies:
flutter:
Expand All @@ -15,10 +23,3 @@ dev_dependencies:
flutter_test:
sdk: flutter
pedantic: ^1.8.0

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.flutter_plugin_android_lifecycle
pluginClass: FlutterAndroidLifecyclePlugin
29 changes: 15 additions & 14 deletions packages/google_maps_flutter/google_maps_flutter/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,22 @@
name: google_maps_flutter
description: A Flutter plugin for integrating Google Maps in iOS and Android applications.
homepage: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter/google_maps_flutter
repository: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter/google_maps_flutter
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+maps%22
version: 2.0.6

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.googlemaps
pluginClass: GoogleMapsPlugin
ios:
pluginClass: FLTGoogleMapsPlugin

dependencies:
flutter:
sdk: flutter
Expand All @@ -21,16 +35,3 @@ dev_dependencies:
pedantic: ^1.10.0
plugin_platform_interface: ^2.0.0
stream_transform: ^2.0.0

flutter:
plugin:
platforms:
android:
package: io.flutter.plugins.googlemaps
pluginClass: GoogleMapsPlugin
ios:
pluginClass: FLTGoogleMapsPlugin

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=1.22.0"
Original file line number Diff line number Diff line change
@@ -1,24 +1,25 @@
name: google_maps_flutter_platform_interface
description: A common platform interface for the google_maps_flutter plugin.
homepage: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter/google_maps_flutter_platform_interface
repository: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter/google_maps_flutter_platform_interface
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+maps%22
# NOTE: We strongly prefer non-breaking changes, even at the expense of a
# less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
version: 2.0.4

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=2.0.0"

dependencies:
collection: ^1.15.0
flutter:
sdk: flutter
meta: ^1.3.0
plugin_platform_interface: ^2.0.0
stream_transform: ^2.0.0
collection: ^1.15.0

dev_dependencies:
flutter_test:
sdk: flutter
mockito: ^5.0.0-nullsafety.0
pedantic: ^1.10.0

environment:
sdk: '>=2.12.0 <3.0.0'
flutter: ">=1.9.1+hotfix.4"
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
name: google_maps_flutter_web
description: Web platform implementation of google_maps_flutter
homepage: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter
repository: https://github.com/flutter/plugins/tree/master/packages/google_maps_flutter
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+maps%22
version: 0.3.0

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=2.0.0"

flutter:
plugin:
platforms:
Expand All @@ -15,17 +20,13 @@ dependencies:
sdk: flutter
flutter_web_plugins:
sdk: flutter
meta: ^1.3.0
google_maps_flutter_platform_interface: ^2.0.1
google_maps: ^5.1.0
stream_transform: ^2.0.0
meta: ^1.3.0
sanitize_html: ^2.0.0
stream_transform: ^2.0.0

dev_dependencies:
flutter_test:
sdk: flutter
pedantic: ^1.10.0

environment:
sdk: ">=2.12.0 <3.0.0"
flutter: ">=1.20.0"
Loading

0 comments on commit 388f916

Please sign in to comment.