Skip to content

Conversation

@Icelk
Copy link
Contributor

@Icelk Icelk commented Mar 28, 2023

This enables socket configuration for the listener.

Copy link
Collaborator

@FrankReh FrankReh left a comment

Choose a reason for hiding this comment

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

Looks reasonable. A couple of questions.

@FrankReh
Copy link
Collaborator

@Icelk You can use cargo test and cargo doc --no-deps to get most tests run locally. Sometimes the failure in CI will be because the CI kernel is 5.15 and not 6.0 or 6.1 but maybe that's not the current case.

@Icelk
Copy link
Contributor Author

Icelk commented Mar 28, 2023

Fixed the test now. Sorry for the CI failure.

@FrankReh FrankReh merged commit ee8318d into tokio-rs:master Mar 29, 2023
@FrankReh
Copy link
Collaborator

Thanks!

@Icelk Icelk deleted the tcp-listener-fd branch March 29, 2023 13:46
@Icelk
Copy link
Contributor Author

Icelk commented Mar 29, 2023

Thank you! You've made it very easy to contribute.

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