Skip to content

Lower the minimum iOS and tvOS versions to 15 - #43

Open
Jeehut wants to merge 1 commit into
mainfrom
wip/ios15-tvos15
Open

Jeehut wants to merge 1 commit into
mainfrom
wip/ios15-tvos15

Conversation

@Jeehut

@Jeehut Jeehut commented Oct 25, 2026

Copy link
Copy Markdown
Member

Summary

Picks up #39 by @H4Hugo, rebased onto current main. His commit is kept as-is, so he stays the author in the history.

-platforms: [.macOS(.v13), .iOS(.v16), .tvOS(.v16), .watchOS(.v9), .macCatalyst(.v16)]
+platforms: [.macOS(.v13), .iOS(.v15), .tvOS(.v15), .watchOS(.v9), .macCatalyst(.v15)]

The log APIs and the Duration helpers need Duration, which arrived in iOS 16 and tvOS 16. They get @available(iOS 16, tvOS 16, *). Everything else in ErrorKit works on 15.

The only conflict was in ErrorKit+OSLog.swift: #42 added a comment above the Duration extension. Both the comment and the new annotation stay.

Supersedes #39.

Evidence

  • Before: main can't be used by apps that support iOS 15 or tvOS 15.
    After:
    • xcodebuild build succeeds for generic/platform=iOS and generic/platform=tvOS with the lower minimums.
    • xcodebuild build-for-testing succeeds for the iOS Simulator.
    • swift test on macOS passes: 21 tests in 5 suites.

CI only runs swift test on macOS, so the iOS and tvOS builds above are the check for the new minimums.

Merge Danger

Door: two-way

Blast Radius: none

Lowering a minimum can't break existing apps. The log APIs are now unavailable on iOS 15 and tvOS 15, which is where they couldn't work anyway.

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