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

Remove required platforms from SPM #761

Merged
merged 1 commit into from
Mar 24, 2020
Merged

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Mar 24, 2020

Motivation:

We recently specified which platforms we supported in Package.swift
based on what SwiftNIO supported. These minimum versions recently
decreased making the platform requirement specification in Package.swift
unnecessary.

Modifications:

  • Remove platforms from Package.swift
  • Fix #file passed to function argument

Result:

  • Dependents of gRPC Swift don't require their Package.swift to declare
    their supported platforms

Motivation:

We recently specified which platforms we supported in Package.swift
based on what SwiftNIO supported. These minimum versions recently
decreased making the platform requirement specification in Package.swift
unnecessary.

Modifications:

- Remove platforms from Package.swift
- Fix #file passed to function argument

Result:

- Dependents of gRPC Swift don't require their Package.swift to declare
  their supported platforms
@glbrntt glbrntt added the nio label Mar 24, 2020
@glbrntt glbrntt requested a review from Lukasa March 24, 2020 12:00
@glbrntt glbrntt added the 🔨 semver/patch No public API change. label Mar 24, 2020
@glbrntt glbrntt merged commit 373ffd5 into grpc:master Mar 24, 2020
@MrMage
Copy link
Collaborator

MrMage commented Mar 24, 2020

What are the new platform requirements for SwiftNIO? I'm curious.

@Lukasa
Copy link
Collaborator

Lukasa commented Mar 24, 2020

They align with the defaults: macOS 10.10, oldest SDKs available in all Swift 5-supporting Xcodes for everything else.

@glbrntt glbrntt deleted the gb-package-versions branch March 24, 2020 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants