Skip to content

Conversation

GuillaumeGomez
Copy link
Member

This is a small cleanup. The goal is to unify a bit things to make the reading simpler.

r? @kinnison

cc @rust-lang/rustdoc

Copy link
Contributor

@kinnison kinnison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this is tidier, I think I'd benefit from a bit of documentation on the purpose and meaning of the arguments to, and results of, the init_lints() function. I appreciate the behaviour doesn't appear to have changed, but that doesn't mean I understand it.

@GuillaumeGomez
Copy link
Member Author

Good point, thanks for bringing it up! I'll document the function to make it as clear as possible.

@GuillaumeGomez
Copy link
Member Author

I added more documentation to explain what's going on.

@crlf0710 crlf0710 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 8, 2020
@kinnison
Copy link
Contributor

kinnison commented May 8, 2020

That doc helps me understand a good bit better, thanks.

@bors r+

@bors
Copy link
Collaborator

bors commented May 8, 2020

📌 Commit 9ae85e1 has been approved by kinnison

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 8, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request May 8, 2020
Rollup of 6 pull requests

Successful merges:

 - rust-lang#71581 (Unify lints handling in rustdoc)
 - rust-lang#71710 (Test for zero-sized function items not ICEing)
 - rust-lang#71970 (Improve bitcode generation for Apple platforms)
 - rust-lang#71975 (Reduce `TypedArena` creations in `check_match`.)
 - rust-lang#72003 (allow wasm target for rustc-ap-rustc_span)
 - rust-lang#72017 (Work around ICEs during cross-compilation for target, ast, & attr)

Failed merges:

r? @ghost
@bors bors merged commit 807e8b8 into rust-lang:master May 8, 2020
@GuillaumeGomez GuillaumeGomez deleted the unify-lints-handling branch May 9, 2020 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants