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

stream: add examples to wrapper types #7024

Merged
merged 5 commits into from
Dec 10, 2024

Conversation

joshka
Copy link
Contributor

@joshka joshka commented Dec 8, 2024

Motivation

I noticed a question on the discord about one of the stream wrappers, and wrote some sample code to answer it. This extends that to some of the other wrappers.

Solution

New doc comment examples in the tokio-stream::wrappers types.

Copy link
Contributor

@Darksonn Darksonn left a comment

Choose a reason for hiding this comment

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

Thanks!

tokio-stream/src/wrappers/broadcast.rs Outdated Show resolved Hide resolved
@Darksonn Darksonn changed the title Master stream: add examples to wrapper types Dec 9, 2024
@Darksonn Darksonn added T-docs Topic: documentation A-tokio-stream Area: The tokio-stream crate labels Dec 9, 2024
@joshka joshka force-pushed the jm/tokio-stream-wrappers-examples branch from 95b81d0 to 75bb441 Compare December 10, 2024 00:49
@joshka joshka force-pushed the jm/tokio-stream-wrappers-examples branch from 75bb441 to 1ae3206 Compare December 10, 2024 01:45
Copy link
Contributor

@Darksonn Darksonn left a comment

Choose a reason for hiding this comment

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

Thanks.

@Darksonn Darksonn enabled auto-merge (squash) December 10, 2024 10:56
@Darksonn Darksonn merged commit 6d15c6c into tokio-rs:master Dec 10, 2024
81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio-stream Area: The tokio-stream crate T-docs Topic: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants