Skip to content

Raise minimum supported Swift version to 5.4 (#1397) #1398

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

Merged

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented May 4, 2022

Motivation:

SwiftNIO raised its minimum supported Swift version to 5.4 in the 2.40.0
release. To be able to use the latest updates, we must do the same.

Modifications:

  • Bump swift-tools-version to 5.4 in the various package manifests
    (including QPS and FuzzTesting)
  • Add excludes for non-source files (READMEs and protos)
  • Use 'executableTarget' instead of 'target' for executables
  • Update the README to reflect supported versions
  • Remove 5.2 and 5.3 from CI
  • Remove LinuxMain.swift as test discovery is enabled by default

Result:

Minimum supported Swift version is 5.4

(cherry picked from commit edcd6b9)

Motivation:

SwiftNIO raised its minimum supported Swift version to 5.4 in the 2.40.0
release. To be able to use the latest updates, we must do the same.

Modifications:

- Bump swift-tools-version to 5.4 in the various package manifests
  (including QPS and FuzzTesting)
- Add excludes for non-source files (READMEs and protos)
- Use 'executableTarget' instead of 'target' for executables
- Update the README to reflect supported versions
- Remove 5.2 and 5.3 from CI
- Remove LinuxMain.swift as test discovery is enabled by default

Result:

Minimum supported Swift version is 5.4

(cherry picked from commit edcd6b9)
@glbrntt glbrntt merged commit 65f7bee into grpc:1.7.1-async-await May 4, 2022
@glbrntt glbrntt deleted the gb-minimum-swift-version-async branch May 4, 2022 10:03
@glbrntt glbrntt added the 🆕 semver/minor Adds new public API. label Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 semver/minor Adds new public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants