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

Drop support for Swift 5.0 #1070

Merged
merged 1 commit into from
Dec 10, 2020
Merged

Drop support for Swift 5.0 #1070

merged 1 commit into from
Dec 10, 2020

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Dec 9, 2020

Motivation:

At this point Swift 5.0 is reasonably old; 5.1 has been around well over
a year and most of the Swift on Server community adopts new Swift versions
pretty quickly. Moreover, you have to go back to Xcode 10 to get a Swift
5.0 compiler on macOS. We should bump the minimum required Swift version
before doing so is a breaking change.

Modifications:

  • Raise minimum version to 5.1
  • Update docs, CI, scaffolding etc.
  • Remove test manifests, use test discovery instead

Result:

@glbrntt glbrntt requested a review from Lukasa December 9, 2020 13:41
@glbrntt glbrntt added the ⚠️ semver/major Breaks existing public API. label Dec 9, 2020
@glbrntt glbrntt force-pushed the gb-swift-support branch 2 times, most recently from f541350 to 77b2040 Compare December 9, 2020 15:38
Motivation:

At this point Swift 5.0 is reasonably old; 5.1 has been around well over
a year and most of the Swift on Server community adopts new Swift versions
pretty quickly. Moreover, you have to go back to Xcode 10 to get a Swift
5.0 compiler on macOS. We should bump the minimum required Swift version
before doing so is a breaking change.

Modifications:

- Raise minimum version to 5.1
- Update docs, CI, scaffolding etc.
- Remove test manifests, use test discovery instead

Result:

- Swift 5.0 is no longer supported
- Resolves grpc#1069
@glbrntt glbrntt merged commit b88c222 into grpc:main Dec 10, 2020
@glbrntt glbrntt deleted the gb-swift-support branch December 10, 2020 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ semver/major Breaks existing public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test with swift 5.3 and Xcode 12.x
3 participants