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

Bump deployment target to iOS 15 #81

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

Bump deployment target to iOS 15 #81

wants to merge 2 commits into from

Conversation

NickEntin
Copy link
Collaborator

@NickEntin NickEntin commented Oct 5, 2024

With the release of Xcode 16 the furthest back you can deploy an app is iOS 15, so there's no real point in supporting prior to that.

Copy link
Collaborator

@dfed dfed left a comment

Choose a reason for hiding this comment

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

LGTM under the assumption that the next release is a breaking change. with some blocking comments. My guess is that we want to move to swift v5.10 everywhere except the swift package version, with a possible intent to move to Swift 6 soonish (though once you only support Xcode 16, moving to swift 6 might not be a breaking change).

Sidebar: TIL that the minimum supported version is iOS 15 now. I gotta go update a few of my repos 😅

@@ -719,7 +719,7 @@
DEBUG_INFORMATION_FORMAT = dwarf;
GCC_C_LANGUAGE_STANDARD = gnu11;
INFOPLIST_FILE = "Performance Tests/Info.plist";
IPHONEOS_DEPLOYMENT_TARGET = 12.2;
IPHONEOS_DEPLOYMENT_TARGET = 15.6;
Copy link
Collaborator

Choose a reason for hiding this comment

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

.6 not .0?

@@ -7,7 +7,7 @@ Pod::Spec.new do |s|
s.author = 'Square'
s.source = { :git => 'https://github.com/CashApp/Stagehand.git', :tag => s.version.to_s }

s.ios.deployment_target = '12.0'
s.ios.deployment_target = '15.0'

s.swift_version = '5.0.1'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We likely want 5.10 or 6.0 here

@@ -21,7 +21,7 @@ import PackageDescription
let package = Package(
Copy link
Collaborator

Choose a reason for hiding this comment

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

We likely want to change the swift tools version to 6.0 at the top of the file, and remove the stray let version line at the bottom of this file

@@ -21,7 +21,7 @@ import PackageDescription
let package = Package(
name: "Stagehand",
platforms: [
.iOS(.v12),
.iOS(.v15),
.macOS(.v11),
],
products: [
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want the swift version of this repo to be 5 or 6? I know we want to support SPM v6, but what language mode do we want for this repo? Are we ready for 6? Or is there more work to do first?

@@ -7,7 +7,7 @@ Pod::Spec.new do |s|
s.author = 'Square'
s.source = { :git => 'https://github.com/CashApp/Stagehand.git', :tag => s.version.to_s }

s.ios.deployment_target = '12.0'
s.ios.deployment_target = '15.0'

s.swift_version = '5.0.1'
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be 5.10 or 6.0

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