Skip to content

Conversation

loic-sharma
Copy link
Member

@loic-sharma loic-sharma commented May 29, 2024

Adds Swift Package Manager compatibility to image_picker_ios.

The previous attempt, #6617, was partially reverted due to flutter/flutter#148307. This reland uses the new approach proposed in flutter/flutter#148572: the Swift Package Manager package does not have a .modulemap. As a result, the plugin no longer has a .Test submodule if using Swift Package Manager.

Fixes flutter/flutter#146919.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@loic-sharma loic-sharma force-pushed the spm_image_picker_v2 branch 2 times, most recently from a5611ce to cf032f1 Compare May 30, 2024 20:30
@loic-sharma loic-sharma marked this pull request as ready for review May 30, 2024 21:05
@loic-sharma loic-sharma requested a review from vashworth as a code owner May 30, 2024 21:05
Copy link
Contributor

@vashworth vashworth left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM.

We could consider adding a TODO comment in Package.swift to add a modulemap with a test module once Cocoapods support is dropped, but since the long-term plan is to migrate everything to Swift anyway I don't feel strongly about it.

@loic-sharma loic-sharma added the autosubmit Merge PR when tree becomes green via auto submit App label May 31, 2024
@auto-submit auto-submit bot merged commit 6872c3b into flutter:main May 31, 2024
@loic-sharma loic-sharma deleted the spm_image_picker_v2 branch May 31, 2024 19:03
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 3, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jun 3, 2024
flutter/packages@d8e8e8c...11e192a

2024-06-01 [email protected] Roll Flutter from c85fa6a to 7eebe29 (26 revisions) (flutter/packages#6836)
2024-05-31 [email protected] [image_picker_ios] Re-adds Swift Package Manager compatibility (flutter/packages#6833)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: image_picker platform-ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[image_picker_ios] Add Swift Package Manager compatibility to image_picker_ios
3 participants