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

Show features (futures) in docs #151

Merged
merged 3 commits into from
May 19, 2022
Merged

Conversation

faern
Copy link
Contributor

@faern faern commented Jun 20, 2020

The fact that there is a futures feature flag and a block_on method was a bit hard to discover. At least for me. That lead me to create #148. Turns out it was not needed.

To make it more discoverable, this PR makes docs.rs build the documentation with all features active, so the extra functionality they enable are visible. It also uses the unstable doc_cfg attribute to make it more obvious in the docs that those items are feature gated.

This is taken more or less directly from how the tokio crate does the same thing.

@hawkw
Copy link
Member

hawkw commented May 19, 2022

It would be really nice to get this merged, but it looks like the PR is quite out of date. I'm going to update it myself.

@hawkw hawkw enabled auto-merge (squash) May 19, 2022 18:49
@hawkw hawkw merged commit 9544793 into tokio-rs:master May 19, 2022
hawkw added a commit that referenced this pull request May 19, 2022
PR #151 added a CI job to try and build the documentation, failing on
RustDoc errors. It turned out there was a handful of broken links, so
that job is now failing. 

I meant to fix this as part of #151 but accidentally automerged it
before fixing the docs, since automerge wasn't configured to require the
docs build job to pass. Whoops.

Signed-off-by: Eliza Weisman <eliza@buoyant.io>
hawkw added a commit that referenced this pull request May 19, 2022
# 0.5.6 (May 19, 2022)

### Added

- cell: add `UnsafeCell::into_inner` for parity with `std` (#272)
- sync: re-enable `Arc::strong_count` (#172)
- sync: implement `Arc::try_unwrap` (#262)
- sync: add `mpsc::Receiver::try_recv` (#262)

### Documented

- show feature flags in docs (#151)
- fix broken RustDoc links (#273)
hawkw added a commit that referenced this pull request May 19, 2022
# 0.5.6 (May 19, 2022)

### Added

- cell: add `UnsafeCell::into_inner` for parity with `std` (#272)
- sync: re-enable `Arc::strong_count` (#172)
- sync: implement `Arc::try_unwrap` (#262)
- sync: add `mpsc::Receiver::try_recv` (#262)

### Documented

- show feature flags in docs (#151)
- fix broken RustDoc links (#273)
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.

None yet

3 participants