Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Replace NIOSendable with Sendable #181

Merged
merged 2 commits into from
Oct 13, 2022
Merged

Replace NIOSendable with Sendable #181

merged 2 commits into from
Oct 13, 2022

Conversation

dnadoba
Copy link
Member

@dnadoba dnadoba commented Oct 13, 2022

We now require Swift 5.5.2 which includes Sendable so we no longer require the NIOSendable workaround.
NIOSendable is just a typealias of Sendable so this is a non source breaking change.
This allows swift-nio to eventually deprecate NIOSendable.

@dnadoba dnadoba added the 🔨 semver/patch No public API change. label Oct 13, 2022
@Lukasa Lukasa enabled auto-merge (squash) October 13, 2022 14:35
@Lukasa Lukasa merged commit d373eaf into apple:main Oct 13, 2022
@dnadoba dnadoba deleted the dn-remove-niosendable branch October 13, 2022 14:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Lukasa Lukasa Lukasa approved these changes

Assignees
No one assigned
Labels
🔨 semver/patch No public API change.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dnadoba @Lukasa