-
Notifications
You must be signed in to change notification settings - Fork 307
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
Xcode 15.3 beta 1
: fix compilation errors
#3599
Conversation
There's a proposal to address this: https://github.com/apple/swift-evolution/blob/9a8f81d75bf58c303d7aa7317a9528931ac7172c/proposals/NNNN-generalize-async-sequence.md But that's for Swift 5.11. We'll need a way to fix this on Swift 5.10. |
Xcode 15.3 beta
: fix compilation errorsXcode 15.3 beta 1
: fix compilation errors
func deferMessagesIfNeeded() async throws { | ||
// `nonisolated` required to work around Swift 5.10 issue. | ||
// See https://github.com/RevenueCat/purchases-ios/pull/3599 | ||
nonisolated func deferMessagesIfNeeded() async throws { |
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.
More context for this: https://twitter.com/nachosoto/status/1750634899810472087?s=61&t=gxxxEpOJpZ2mMeKVPUPKew
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.
SHIP IT
**This is an automatic release.** ### Bugfixes * `Xcode 15.3 beta 1`: fix compilation errors (#3599) via NachoSoto (@NachoSoto) ### Other Changes * `Xcode 15.3 beta 1`: fix warnings on tests (#3600) via NachoSoto (@NachoSoto)
Follow up to #3599 and #3600. These `@MainActor` annotations don't work anymore because of this: > Main actor-isolated class '...' has different actor isolation from nonisolated superclass 'TestCase'; this is an error in Swift 6 They were necessary with an older version of Xcode (likely 13.x), but not anymore.
See #3599. This is listed in the [release notes](https://developer.apple.com/documentation/xcode-release-notes/xcode-15_3-release-notes) as fixed.
See #3599. This is listed in the [release notes](https://developer.apple.com/documentation/xcode-release-notes/xcode-15_3-release-notes) as fixed.
Follow up to #3599 and #3600. These `@MainActor` annotations don't work anymore because of this: > Main actor-isolated class '...' has different actor isolation from nonisolated superclass 'TestCase'; this is an error in Swift 6 They were necessary with an older version of Xcode (likely 13.x), but not anymore.
Follow up to #3599 and #3600. These `@MainActor` annotations don't work anymore because of this: > Main actor-isolated class '...' has different actor isolation from nonisolated superclass 'TestCase'; this is an error in Swift 6 They were necessary with an older version of Xcode (likely 13.x), but not anymore.
Follow up to #3599 and #3600. These `@MainActor` annotations don't work anymore because of this: > Main actor-isolated class '...' has different actor isolation from nonisolated superclass 'TestCase'; this is an error in Swift 6 They were necessary with an older version of Xcode (likely 13.x), but not anymore. --------- Co-authored-by: Mark Villacampa <m@markvillacampa.com>
Changes:
InternalAPI
is nowSendable
to deal with new compilation errorPaywallEventStoreType
is nowSendable
SK2BeginRefundRequestHelperType
to makeSK2BeginRefundRequestHelper
Sendable
MockSK2BeginRefundRequestHelper
to be thread-safe andSendable
DebugContentViews
DebugViewModel
StoreMessagesHelper