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

task: fix unstable API documentation notes #4503

Merged
merged 1 commit into from
Feb 15, 2022
Merged

Conversation

hawkw
Copy link
Member

@hawkw hawkw commented Feb 15, 2022

Motivation

PR #4499 made the JoinSet API unstable, but did not add a
documentation note explaining unstable features. In general, since the
docs.rs build includes unstable APIs, it's probably worth including
these notes so that users understand what it means for an API to be
unstable.

Solution

This branch adds a note on unstable APIs to the JoinSet type-level
documentation, similar to the notes for task::Builder and the runtime
metrics APIs.

Also, I noticed that there was a broken link to the top-level
documentation on unstable APIs in the docs for task::Builder, so I
fixed that as well.

PR #4499 made the `JoinSet` API unstable, but did not add a
documentation note explaining unstable features. This branch adds a note
on unstable APIs to the `JoinSet` type-level documentation, similar to
the notes for `task::Builder` and the runtime metrics APIs.

Also, I noticed that there was a broken link to the top-level
documentation on unstable APIs in the docs for `task::Builder`, so I
fixed that as well.
@hawkw hawkw requested a review from Darksonn February 15, 2022 17:24
@hawkw hawkw added A-tokio Area: The main tokio crate T-docs Topic: documentation M-task Module: tokio/task labels Feb 15, 2022
@hawkw hawkw merged commit 8758965 into master Feb 15, 2022
@hawkw hawkw deleted the eliza/unstable-joinset-docs branch February 15, 2022 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate M-task Module: tokio/task T-docs Topic: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants