-
Notifications
You must be signed in to change notification settings - Fork 290
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
[main] Bump to Magnes v5.5.0 #1217
Conversation
…ile exists in App Archive
Co-authored-by: Sammy Cannillo <scannillo@paypal.com>
@@ -1,6 +1,6 @@ | |||
#if 0 | |||
#elif defined(__arm64__) && __arm64__ | |||
// Generated by Apple Swift version 5.8 (swiftlang-5.8.0.124.2 clang-1403.0.22.11.100) | |||
// Generated by Apple Swift version 5.7.1 (swiftlang-5.7.1.135.3 clang-1400.0.29.51) |
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.
I think this is fine since it's backward compatible with merchant apps that may use newer Swift versions. Please call out if you disagree!
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.
I wonder if this will be an issue with this upcoming requirement - https://developer.apple.com/news/upcoming-requirements/?id=04292024a. I don't think so, but good to discuss
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.
I don't think it should be an issue as long as it works currently. It looks like Xcode 15 supports down to Swift 4
@@ -1,6 +1,6 @@ | |||
#if 0 | |||
#elif defined(__arm64__) && __arm64__ | |||
// Generated by Apple Swift version 5.8 (swiftlang-5.8.0.124.2 clang-1403.0.22.11.100) | |||
// Generated by Apple Swift version 5.7.1 (swiftlang-5.7.1.135.3 clang-1400.0.29.51) |
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.
I don't think it should be an issue as long as it works currently. It looks like Xcode 15 supports down to Swift 4
Summary of changes
Checklist
Authors
@scannillo