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

Enable futures feature for docs.rs builds #35

Merged
merged 1 commit into from
Oct 1, 2023

Conversation

Swatinem
Copy link
Contributor

@Swatinem Swatinem commented Oct 1, 2023

The docs at docs.rs do not include AsyncStreamCDC. This fixes that by making sure the futures feature is enabled. I picked that feature as it has less dependencies, and --all-features would mean that as_stream would not be included, as the current cfg flags output neither version when both features are enabled.

The docs at docs.rs do not include `AsyncStreamCDC`. This fixes that by making sure the `futures` feature is enabled. I picked that feature as it has less dependencies, and `--all-features` would mean that `as_stream` would not be included, as the current `cfg` flags output neither version when both features are enabled.
@nlfiedler nlfiedler merged commit 8653138 into nlfiedler:master Oct 1, 2023
3 checks passed
@nlfiedler
Copy link
Owner

Terrific, thank you for that.

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