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

Adopt concurrency adoption guidelines #230

Merged
merged 2 commits into from
Dec 6, 2021

Conversation

stevapple
Copy link
Contributor

@stevapple stevapple commented Sep 23, 2021

Adopt Swift Concurrency adoption guidelines for Swift Server Libraries (swift-server/guides#70).

Motivation:

Adopt Swift Concurrency adoption guidelines (for the correct way to import _Concurrency).

Modifications:

  • Use #if compiler(>=5.5) && canImport(_Concurrency) to judge if Concurrency is available;
  • Some cleaning.

@swift-server-bot
Copy link

Can one of the admins verify this patch?

6 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@stevapple stevapple marked this pull request as draft September 26, 2021 23:57
@stevapple stevapple force-pushed the concurrency-guideline branch 3 times, most recently from b91487c to a844d9e Compare December 6, 2021 09:49
@stevapple stevapple marked this pull request as ready for review December 6, 2021 09:49
@stevapple stevapple force-pushed the concurrency-guideline branch 2 times, most recently from 035c5ef to 0a0d1aa Compare December 6, 2021 09:57
@stevapple
Copy link
Contributor Author

cc @tomerd @fabianfett

Copy link
Member

@fabianfett fabianfett left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @stevapple

@fabianfett
Copy link
Member

@swift-server-bot test this please

@fabianfett
Copy link
Member

@stevapple Let's do the sendable checks in another pr please! We don't mind them for merging right now.

@fabianfett
Copy link
Member

@swift-server-bot test this please (@tomerd @yim-lee could I get bot rights for this repo and lambda-events as well please?)

@yim-lee
Copy link
Contributor

yim-lee commented Dec 6, 2021

could I get bot rights for this repo and lambda-events as well please?

@fabianfett I believe you have them already. Are you experiencing issues?

@fabianfett fabianfett added the patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1) label Dec 6, 2021
@fabianfett
Copy link
Member

Nightly failed because of Sendable checking.

@fabianfett fabianfett merged commit 4317c53 into swift-server:main Dec 6, 2021
@stevapple stevapple deleted the concurrency-guideline branch December 7, 2021 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants