Skip to content

Conversation

@finagolfin
Copy link
Contributor

Android doesn't have getpass() either.

@Joannis, submitted as you asked for.

Android doesn't have getpass() either.
@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?

Copy link
Contributor

@Lukasa Lukasa 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!

@Lukasa
Copy link
Contributor

Lukasa commented Feb 6, 2023

@swift-server-bot add to allowlist

@Lukasa Lukasa added the semver/none No version bump required. label Feb 6, 2023
@Lukasa Lukasa enabled auto-merge (squash) February 6, 2023 08:38
@finagolfin
Copy link
Contributor Author

CI failures are unrelated.

@Lukasa
Copy link
Contributor

Lukasa commented Feb 6, 2023

They are indeed: we reduced allocations in NIO so the tests need to be updated. I'll put up a PR shortly.

@Lukasa
Copy link
Contributor

Lukasa commented Feb 6, 2023

See #135.

@Lukasa Lukasa merged commit 28d80d4 into apple:main Feb 6, 2023
@finagolfin finagolfin deleted the droid branch February 13, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants