Skip to content
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

Obvious Fix: Change minimum iOS version from 15 to 13 where possible #145

Conversation

sebastiangrail
Copy link
Contributor

All async/await APIs are currently only available for iOS 15 but the minimum requirement for async/await is iOS 13. This PR changes the minimum iOS version from 15 to 13 where possible.

@mikenachbaur-okta
Copy link
Contributor

Thank you for submitting this. The minimum was set to iOS 15 due to issues with Xcode 13 throwing compiler errors when Swift Concurrency features were used while targeting older iOS versions. These problems seem to have been fixed in Xcode 14, so we can certainly go ahead with this.

Since this SDK supports platforms other than iOS (tvOS, watchOS, and macOS), we'd need to ensure the supported versions match all available platforms though.

@sebastiangrail would you be able to update the supported versions in those @available attributes to support tvOS 13, watchOS 6, and macOS 10.15?

@sebastiangrail
Copy link
Contributor Author

@mikenachbaur-okta Done. Note that the build for tvOS and watchOS is broken for unrelated reasons (tested on Xcode 14.3.1).

@mikenachbaur-okta mikenachbaur-okta changed the base branch from master to man-contributions August 1, 2023 19:49
@mikenachbaur-okta mikenachbaur-okta merged commit f391358 into okta:man-contributions Aug 1, 2023
4 of 10 checks passed
mikenachbaur-okta added a commit that referenced this pull request Aug 1, 2023
@sebastiangrail sebastiangrail deleted the fix-minimum-iOS-requirement branch August 7, 2023 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants