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

fix(client/ios): update minimum deployment target to 15.5 #2309

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sbruens
Copy link
Contributor

@sbruens sbruens commented Dec 10, 2024

Minimum deployment version in the XCode project was already upgraded in #2288. This PR updates some trailing instances to match the version set in the XCode project.

iOS 15 recently reached end of life. Anything older reached end of life at least 1 year ago: https://endoflife.date/ios

@sbruens sbruens requested review from a team as code owners December 10, 2024 15:54
@@ -24,7 +24,7 @@ let package = Package(
// CocoaLumberjack 3.8.0 dropped support for iOS < 11 and macOS < 10.13.
// See https://github.com/CocoaLumberjack/CocoaLumberjack/releases/tag/3.8.0.
// These cannot be upgraded without also upgrading the entire project.
.iOS(.v13),
.iOS("15.5"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can also use .15_5, but I guess it doesn't matter.

Copy link
Contributor Author

@sbruens sbruens Dec 10, 2024

Choose a reason for hiding this comment

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

That's what I thought but you can't. I had it in a previous commit, but it failed to build:

  /Users/runner/work/outline-apps/outline-apps/client/src/cordova/apple/OutlineAppleLib/Package.swift:28:14: error: reference to member 'v15_5' cannot be resolved without a contextual type
         .iOS(.v15_5),

Turns out only v15 exists: http://go/appledoc/packagedescription/supportedplatform/iosversion/v15. I guess only the v10_X macOS versions specified more granular version types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants