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

Canonicalize style of a few doc comments #52258

Closed
wants to merge 1 commit into from

Conversation

steveklabnik
Copy link
Member

@steveklabnik steveklabnik commented Jul 11, 2018

Originally, I noticed these because they used to warn; the warning was fixed, but the comments were in the unidiomatic style, so I re-did them and changed the flow slightly too.

@rust-highfive
Copy link
Collaborator

r? @joshtriplett

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 11, 2018
@steveklabnik steveklabnik changed the title Fix a few documentation warnings Canonicalize style of a few doc comments Jul 11, 2018
@euclio
Copy link
Contributor

euclio commented Jul 11, 2018

#52164 already fixes this.

@joshtriplett
Copy link
Member

@euclio Yeah, looks that way. @steveklabnik, can you confirm, and either close or rebase as appropriate?

@steveklabnik
Copy link
Member Author

That's fine too; let's take that one. Thanks/sorry all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants