Skip to content
This repository was archived by the owner on Dec 3, 2025. It is now read-only.

Conversation

@stuartmorgan-g
Copy link
Collaborator

Updates the example, testbed, and plugins on macOS to account for
flutter/flutter#33997

example_plugin is now entirely Swift, with the ObjC wrapper removed; the
other plugins remain ObjC for now, with a Swift wrapper. The podspecs
have been adjusted to remove the public ObjC headers.

The example has been switched from an empty ObjC registrant to an empty
Swift registrant, and testbed has been updated with the results of
re-running flutter run with the new tooling support, and switched to
calling the Swift registrant as well.

While changing the plugin podspecs, updated them to 10.12 to match the
current FlutterMacOS target.

Updates the example, testbed, and plugins on macOS to account for
flutter/flutter#33997

example_plugin is now entirely Swift, with the ObjC wrapper removed; the
other plugins remain ObjC for now, with a Swift wrapper. The podspecs
have been adjusted to remove the public ObjC headers.

The example has been switched from an empty ObjC registrant to an empty
Swift registrant, and testbed has been updated with the results of
re-running `flutter run` with the new tooling support, and switched to
calling the Swift registrant as well.
@stuartmorgan-g
Copy link
Collaborator Author

stuartmorgan-g commented Jun 6, 2019

/cc @jmagman

Copy link

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM!

@stuartmorgan-g stuartmorgan-g merged commit 4f9aec8 into google:master Jun 6, 2019
@stuartmorgan-g stuartmorgan-g deleted the swift-plugin-conversion branch June 6, 2019 22:49
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants