-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[Swift Package Manager] Update docs after 3.24 release #11071
Conversation
@@ -25,9 +25,11 @@ Using the Flutter CLI to run an app [migrates the project][addSPM] to add | |||
Swift Package Manager integration. | |||
This makes your project download the Swift packages that | |||
your Flutter plugins depend on. | |||
If you disable the Swift Package Manager feature, | |||
you will need to [remove Swift Package Manager integration][removeSPM] from apps | |||
that are migrated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The previous statement was incorrect. You had to undo the integration if you switched to the stable branch (which also implicitly disables the SPM feature). If you disabled the SPM feature but stayed on the master channel, the app would continue to build fine. I had misunderstood how this worked!
Visit the preview URL for this PR (updated for commit 6be62c1): https://flutter-docs-prod--pr11071-spm-flutter-3-24-pqg90v64.web.app |
@@ -440,8 +440,8 @@ The example below uses `ios`, replace `ios` with `macos`/`darwin` as applicable. | |||
|
|||
**Do not commit the migration's changes to your version control system.** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, I kept this recommendation to not check-in the SPM integration for your example app. This ensures customers using Flutter version lower than 3.24 can continue to build the example app.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Prior to Flutter 3.24, a Flutter app that was migrated to have Swift Package Manager integration could not be built on the stable branch.
With Flutter 3.24, the app will now build successfully. However, plugins will continue to be installed using CocoaPods as the SwiftPM feature remains unavailable on the stable channel.
Issues fixed by this PR (if any): flutter/flutter#153448
PRs or commits this PR depends on (if any):
Presubmit checklist