-
Notifications
You must be signed in to change notification settings - Fork 27
Add Support for Swift Concurrency #201
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
Comments
Hi @Mackarous - this is planned but we do not yet have an ETA. |
@crleona Xcode 16 is around the corner, any plans to get this done? |
please support it |
@klioop I looked into originally trying to support this when I needed it last year. There's no way they'll end up doing this. It would require a thorough understanding of Swift Concurrency throughout the codebase and almost an entire rewrite of how things are architected at the moment as they utilize a lot of APIs that Apple has marked as You are better off just using |
What can we do if With 1.0.0 we did not get that issue, with 1.12.0 we have that. |
Summary
Currently, if using the SDK in a project that enables strict concurrency checking, Xcode will complain that the
Amplitude
object is not marked asSendable
and is not concurrency safe.To circumvent these warnings (which may become errors in Swift 6), we can mark the library as
@preconcurrency
Motivations
Swift Concurrency is becoming a large part of the Swift language, and moving into 6.0 will be important to add support. This might mean exposing a different kind of object, or migrating the
Amplitude
class itself to an actor.The text was updated successfully, but these errors were encountered: