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

Update supported Swift/Xcode Versions. #1438

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

thomasvl
Copy link
Collaborator

Update the documentation to reflect that we will try to support the older of the Swift NIO supported version and the minimum Xcode version supported for the Apple AppStore.

As of today, that means:

Xcode 14.1 included Swift 5.7. So we'll support Swift 5.6+, but the Apple deployment minimums are moved up to the values from Xcode 14.1.

In addition to the documentation updates, this commit also:

  • Updates the main Package.swift and SwiftProtobuf.podspec for the new deployment minimums.
  • Updates the CI setup to the minimum Swift version.

NOTE: There are no code changes in this commit, once this CL is landed, the code changes will be done in follow ups since there are generation changes.

README.md Outdated Show resolved Hide resolved
Package.swift Outdated Show resolved Hide resolved
Update the documentation to reflect that we will try to support the older of the
Swift NIO supported version and the minimum Xcode version supported for the
Apple AppStore.

As of today, that means:

- Swift NIO: Swift 5.6 - https://github.com/apple/swift-nio#swiftnio-2
- AppStore: Xcode 14.1 - https://developer.apple.com/news/?id=jd9wcyov#:~:text=Starting%20April%2025%2C%202023%2C%20iOS,or%20later.

Xcode 14.1 included Swift 5.7. So we'll support Swift 5.6+, but the Apple
deployment minimums are moved up to the values from Xcode 14.1.

In addition to the documentation updates, this commit also:

- Updates the main Package.swift and SwiftProtobuf.podspec for the new
  deployment minimums.
- Updates the CI setup to the minimum Swift version.

NOTE: There are no code changes in this commit, once this CL is landed, the code
changes will be done in follow ups since there are generation changes.
Copy link
Collaborator

@tbkka tbkka left a comment

Choose a reason for hiding this comment

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

Looks good!

@thomasvl thomasvl merged commit 292d6ac into apple:main Aug 15, 2023
@thomasvl thomasvl deleted the required_versions branch August 15, 2023 14:35
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.

4 participants