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

conversion of doc comments to indicative mood

Merged
merged 1 commit into from Oct 19, 2021

Conversation

antogilbert
Copy link
Contributor

@antogilbert antogilbert commented Oct 17, 2021

Conversion of doc comments from imperative to
indicative mood. Addition of trailing periods
as well.

Fixes: #4074

Motivation

As explained in Issue #4074:
tokio's doc comments are written in a mixture of the indicative mood and imperative mood. They should all be in the indicative mood (e.g. "Returns foo"), to match the stdlib and the standard rustdoc conventions.

Solution

Adapt the doc comments of the main crate to use indicative mood. In addition missing trailing periods were added.

@Darksonn Darksonn added A-tokio T-docs labels Oct 18, 2021
Copy link
Contributor

@Darksonn Darksonn left a comment

I haven't been through everything yet, but a few comments:

tokio/src/io/driver/scheduled_io.rs Outdated Show resolved Hide resolved
tokio/src/loom/std/mutex.rs Outdated Show resolved Hide resolved
tokio/src/runtime/basic_scheduler.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@Darksonn Darksonn left a comment

Thanks for taking the time to go through the entirety of Tokio!

tokio/src/time/driver/wheel/mod.rs Outdated Show resolved Hide resolved
Conversion of doc comments from imperative to
indicative mood. Addition of trailing periods
as well.

Fixes: tokio-rs#4074
@Darksonn Darksonn merged commit 03969cd into tokio-rs:master Oct 19, 2021
27 checks passed
oliver-giersch pushed a commit to oliver-giersch/tokio that referenced this issue Oct 28, 2021
Co-authored-by: Antonello Palazzi <antonello.palazzi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio T-docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants