-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[various] Update example apps to Swift #9347
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
Conversation
CHANGELOG override: build details of example apps are not relevant to clients. |
Version override: The |
@jmagman / @vashworth Do you want me to wait for someone on the iOS team to review this, or should I just land it? |
_exampleApp = nil; | ||
} | ||
|
||
- (void)testQuickActionWithFreshStart { |
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.
It looks like this shouldn't be deleted
How are the PickerSaveImageToPathOperationTests.m tests passing with the images being deleted? Or is it not running any more? |
@@ -88,16 +91,9 @@ | |||
ReferencedContainer = "container:Runner.xcodeproj"> | |||
</BuildableReference> | |||
</BuildableProductRunnable> | |||
<AdditionalOptions> | |||
<AdditionalOption | |||
key = "NSZombieEnabled" |
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.
Oops! This has been here since the plugin was first checked in.
// the unit tests, so it is better to skip the AppDelegate when running unit tests. | ||
BOOL isTesting = NSClassFromString(@"XCTestCase") != nil; | ||
return UIApplicationMain(argc, argv, nil, | ||
isTesting ? nil : NSStringFromClass([AppDelegate class])); |
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.
All of this logic is for XCTest, and those tests moved to camera_avfoundation
when the plugin was fully federated, so this wasn't doing anything. camera_avfoundation
still has this code.
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.
This has been probably-accidentally unused since #466 recreated the example app. Since this isn't the recommended way to configure Google Sign In on iOS anymore, rather than fix it here I'm just deleting the unused file. If in another 3 years someone needs it set up, they can configure it in Info.plist.
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.
These image files are all used in the XCTest tests that moved to image_picker_ios
during full federation, so these were cruft.
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.
These tests were copied to path_provider_foundation
during full federation, and were supposed to be deleted from here at the time to make it a move. This duplicate copy doesn't run because there's no native code in path_provider
, so repo tooling doesn't attempt to run native unit tests.
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.
Same as path_provider; these test files should have been deleted when they were moved to quick_actions_ios.
The explanatory comments that I wrote when I posted the PR and then accidentally left as drafts for several weeks cover this 🤦🏻 |
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, thank you @stuartmorgan-g!
flutter/packages@6d3aaf4...c5ab57a 2025-06-13 [email protected] [flutter_svg] feat: Expose the renderStrategy property in SvgPicture (flutter/packages#9373) 2025-06-12 [email protected] [go_router] Add routing functions to GoRouteData (flutter/packages#9277) 2025-06-12 [email protected] [various] Update example apps to Swift (flutter/packages#9347) 2025-06-12 [email protected] Roll Flutter from 824868f to f79452e (94 revisions) (flutter/packages#9419) 2025-06-12 [email protected] [android_camera_camerax] Fix incorrect camera mirroring for front cameras on devices using `ImageReader` Impeller backend (flutter/packages#9233) 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] 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
Converts the Objective-C iOS example apps for all non-iOS-plugin-implementation packages to Swift.
The process for this was (all using current
stable
, 3.32):flutter create --platforms=ios .
.ios/Runner/Assets.xcassets/AppIcon.appiconset
(this doesn't really matter either way, I just decided not to thrash a bunch of icons, since there's no real reason to and it would have added a ton more files).ios/
, except for.gitignore
which we may as well have a current version of.RunnerTest
target, its source file, and thePodfile
adjustment toRunnerTest
.io.flutter.*
org the examples were first created with.Info.plist
.One-off changes:
Configuration.storkekit
in in_app_purchase.I did not convert the implementation packages in this PR because those will need to be handled more carefully, due to the presence of RunnerTest (and sometimes RunnerUITest) targets that need to be preserved.
Part of flutter/flutter#148586
Pre-Review Checklist
[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or I have commented below to indicate which version change exemption this PR falls under1.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style, or I have commented below to indicate which CHANGELOG exemption this PR falls under1.///
).Footnotes
Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩2 ↩3