8000 Implement plugin tooling support for macOS by stuartmorgan-g · Pull Request #33636 · flutter/flutter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Implement plugin tooling support for macOS #33636

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 31, 2019

Conversation

stuartmorgan-g
Copy link
Contributor

Description

Enables the CocoaPods-based plugin workflow for macOS. This allows a
macOS project to automatically fetch and add native plugin
implementations via CocoaPods for anything in pubspec.yaml, as is done
on iOS.

Related Issues

Fixes #32718

Tests

I added the following tests: New test for macOS-specific branch in cocoapods.dart.

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I updated/added relevant documentation (doc comments with ///).
  • All existing and new tests are passing.
  • The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require Flutter developers to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
  • No, this is not a breaking change.

Enables the CocoaPods-based plugin workflow for macOS. This allows a
macOS project to automatically fetch and add native plugin
implementations via CocoaPods for anything in pubspec.yaml, as is done
on iOS.
@Piinks Piinks added e: desktop tool Affects the "flutter" command-line tool. See also t: labels. a: desktop Running on desktop platform-mac Building on or for macOS specifically and removed e: desktop labels May 31, 2019
Copy link
Member
@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.

LGTM

@stuartmorgan-g stuartmorgan-g merged commit 3ebebeb into flutter:master May 31, 2019
stuartmorgan-g added a commit to google/flutter-desktop-embedding that referenced this pull request May 31, 2019
First half of updating for flutter/flutter#33636

For example:
- Switched from placeholder PluginRegistrant to empty.
- GeneratedPluginRegistrant, as with a default iOS project.

For testbed:
- Same change as to example/ as a starting point.
- Removed the direct dependency on example_plugin.
- Ran 'flutter run', which created the rest of the changes.

For example_plugin:
- Removed the Xcode project and framework-related files.
- Updated structure and file naming to match iOS plugin format.
- Added a podfile.

For all other plugins:
- Commented out macosPrefix so that the new tooling won't
  attempt to treat them as pods and fail.

The second half will be updating the other plugins to pods;
that's left as a follow-up to limit the scope of this patch.
kiku-jw pushed a commit to kiku-jw/flutter that referenced this pull request Jun 14, 2019
Enables the CocoaPods-based plugin workflow for macOS. This allows a
macOS project to automatically fetch and add native plugin
implementations via CocoaPods for anything in pubspec.yaml, as is done
on iOS.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a: desktop Running on desktop platform-mac Building on or for macOS specifically tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add macOS plugin tooling
4 participants
0