Skip to content

Conversation

@FranzBusch
Copy link
Member

Motivation

We should import the correct NIO submodules instead of the umbrella.

Modification

Fix up import statements.

Result

We are now importing the proper submodules

@swift-server-bot
Copy link

Can one of the admins verify this patch?

7 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?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@FranzBusch FranzBusch requested a review from glbrntt March 29, 2022 14:46
# Motivation
We should import the correct NIO submodules instead of the umbrella.

# Modification
Fix up import statements

# Result
We are now importing the proper submodules
@glbrntt
Copy link
Contributor

glbrntt commented Mar 29, 2022

@swift-server-bot add to allowlist

@glbrntt
Copy link
Contributor

glbrntt commented Mar 29, 2022

@swift-server-bot test this please

@yim-lee
Copy link
Member

yim-lee commented Mar 29, 2022

@swift-server-bot test this please

1 similar comment
@FranzBusch
Copy link
Member Author

@swift-server-bot test this please

@glbrntt
Copy link
Contributor

glbrntt commented Mar 30, 2022

@swift-server-bot test this please

@glbrntt
Copy link
Contributor

glbrntt commented Mar 30, 2022

@swift-server-bot add to allowlist

@FranzBusch FranzBusch enabled auto-merge (squash) March 30, 2022 12:50
@FranzBusch FranzBusch merged commit dfd4588 into apple:main Mar 30, 2022
@FranzBusch FranzBusch deleted the import branch March 30, 2022 13:33
@Lukasa Lukasa added the 🔨 semver/patch No public API change. label Apr 4, 2022
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.

6 participants