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

Reduce minimum os requirement and increase swift compiler version #1346

Closed
wants to merge 1 commit into from

Conversation

patskovn
Copy link
Contributor

With Xcode 13.2 we can use swift concurrency on older OSes. I followed swift-nio's implementation of this and adopted it in grpc-swift.
Tested with Xcode 13.0 and Xcode 13.2. Also tested in my project that use both Server and Client implementations. Everything compiles and runs.

@linux-foundation-easycla
Copy link

CLA Not Signed

@glbrntt
Copy link
Collaborator

glbrntt commented Jan 27, 2022

Thanks for the PR @Nekitosss -- however, you'll need to sign the CLA to contribute to the project.

@patskovn
Copy link
Contributor Author

Yeah, I know, sorry. Need to make this official with my employer :) Trying to do this asap

@patskovn
Copy link
Contributor Author

patskovn commented Feb 2, 2022

Update:
My change was approved inside the company. Trying to resolve official bureaucracy stuff and find someone who can be my CLA manager. Hope this will be done soon

@patskovn
Copy link
Contributor Author

patskovn commented Feb 8, 2022

Ok, I cannot sign this CLA. After succes s signing in DocuSign, I redirected into my previous pull request to this repo #1272
and this commit remains unsigned. Sorry

@patskovn patskovn closed this Feb 8, 2022
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