Skip to content

Fixed Xcode 13.2.x / Swift 5.5 compatibility#1994

Merged
NachoSoto merged 1 commit into
mainfrom
xcode-13.2.1-compatibility
Oct 24, 2022
Merged

Fixed Xcode 13.2.x / Swift 5.5 compatibility#1994
NachoSoto merged 1 commit into
mainfrom
xcode-13.2.1-compatibility

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Oct 24, 2022

Copy link
Copy Markdown
Contributor

Fixes SDKONCALL-156.

This will be covered by #1990.

@NachoSoto NachoSoto added the pr:fix A bug fix label Oct 24, 2022
@NachoSoto NachoSoto requested a review from a team October 24, 2022 20:03
@NachoSoto NachoSoto changed the title Fixed Xcode 13.2.x compatibility Fixed Xcode 13.2.x / Swift 5.5 compatibility Oct 24, 2022
Comment thread Sources/Identity/IdentityManager.swift Outdated
Comment thread Sources/Purchasing/Purchases/PurchasesOrchestrator.swift Outdated
Comment thread Sources/Networking/Caching/CustomerInfoCallback.swift Outdated
@aboedo aboedo self-requested a review October 24, 2022 20:32
@NachoSoto NachoSoto force-pushed the xcode-13.2.1-compatibility branch from 1032c14 to 9bc4d8c Compare October 24, 2022 20:50
@NachoSoto NachoSoto force-pushed the xcode-13.2.1-compatibility branch from 9bc4d8c to 8a0efd4 Compare October 24, 2022 20:56

}

extension CustomerAPI: Sendable {}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was unnecessary and fixed a lot of unnecessary warnings.

@NachoSoto NachoSoto merged commit b93b082 into main Oct 24, 2022
@NachoSoto NachoSoto deleted the xcode-13.2.1-compatibility branch October 24, 2022 21:36
NachoSoto pushed a commit that referenced this pull request Oct 26, 2022
**This is an automatic release.**

### Bugfixes
* Fixed Xcode 13.2.x / Swift 5.5 compatibility (#1994) via NachoSoto
(@NachoSoto)
### Other Changes
* Update `fastlane` (#1998) via NachoSoto (@NachoSoto)
* Documentation: fixed missing docs from inherited symbols (#1997) via
NachoSoto (@NachoSoto)
* CI: added job to test compilation with `Xcode 13.2.1` / `Swift 5.5`
(#1990) via NachoSoto (@NachoSoto)
* Extracted `TrialOrIntroPriceEligibilityCheckerType` (#1983) via
NachoSoto (@NachoSoto)
* CI: removed redundant `swiftlint` installation (#1993) via NachoSoto
(@NachoSoto)
* `Nimble`: use a fixed version (#1991) via NachoSoto (@NachoSoto)
* Update fastlane-plugin-revenuecat_internal (#1989) via Cesar de la
Vega (@vegaro)
* `Purchases.logIn`: log warning if attempting to use a static
`appUserID` (#1958) via NachoSoto (@NachoSoto)
* Created `InternalAPI` for "health" request (#1971) via NachoSoto
(@NachoSoto)

Co-authored-by: RCGitBot <dev+RCGitBot@revenuecat.com>
NachoSoto added a commit that referenced this pull request Feb 17, 2023
Fixes #2291.

This has led to issues in the past too (#1994, #2221, 	RevenueCat/purchases-flutter#516, RevenueCat/purchases-flutter#507) because I tried to reuse that code to call a single completion method.
The new version of Swift makes that impossible now, because it detects that `@MainActor` isn't available in iOS 12.

To finally work around all those issues, this simply duplicates this single line of code, which should work across all versions of Swift.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants