Drop iOS/tvOS 9-11, watchOS 3-4, macOS 10.10-10.13 support#1605
Merged
designatednerd merged 7 commits intomainfrom Jan 11, 2021
Merged
Drop iOS/tvOS 9-11, watchOS 3-4, macOS 10.10-10.13 support#1605designatednerd merged 7 commits intomainfrom
designatednerd merged 7 commits intomainfrom
Conversation
…ything to 2018 relases
martijnwalraven
approved these changes
Jan 5, 2021
Contributor
martijnwalraven
left a comment
martijnwalraven
left a comment
There was a problem hiding this comment.
That cleans up nicely!
From what I can see this looks good. I wish we didn't have to duplicate configuration across Xcode project, Cocoapods and Swift package manager. But unfortunately that's the reality of the ecosystem right now.
Contributor
Author
|
Yeah, the duplication is annoying, but given how infrequently this changes, I'm kinda fine with it. |
Contributor
Author
|
OK i'm gonna leave this up until Thursday morning my time for objections, otherwise we'll merge it in! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One of the clearest results of the 2020 developer survey is that our users aren't supporting iOS 9 and 10 anymore. Dropping these two operating systems and their associated versions of tvOS, macOS, and watchOS allowed us to get rid of basically every availability check we currently have. 🎉
BUT, I think we can go one better: iOS 12 supports all the same devices iOS 11 does, so I think we can also drop iOS 11 support. We only had one person who mentioned having to support 11, and their stated reason was to support older devices - but all older devices that support 11 also support 12.
This also lines up with Apple's "Hey there, you sure you don't want to make 12.0 your min SDK?" prompt given every time Xcode 12 is updated. Personally, I'd love to drop 12 too to use Combine within the SDK, but over 25% of our users still have iOS 12 as their min SDK, so I don't think that's realistic for at least another year.
This would leave our min SDKs as:
iOS/tvOS: 12.0
watchOS: 5.0
macOS: 10.14
If you have strong objections to dropping any of these operating systems, please register them in the comments below!