-
Notifications
You must be signed in to change notification settings - Fork 13.2k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Check implementing type for #[doc(hidden)]
#89987
Conversation
r? @jackh726 (rust-highfive has picked a reviewer for you, use r? to override) |
a599b83
to
bc5ed6e
Compare
r? rust-lang/rustdoc |
Apart from the nit, looks good to me, thanks for the improvement/fix! Feel free to r+ if you think the suggestion I made isn't worth it. |
@GuillaumeGomez Is it alright if I like the original code a little better? 😇 (Also, I don't have |
It's perfectly fine. :) Then let's go with this this! @bors: r+ squash |
📌 Commit 8f2551d0dbdd066914ece68d312038a957d6eeea has been approved by |
8f2551d
to
d39a1be
Compare
See rust-lang/homu#136 and rust-lang/homu#158 for more information. |
Done. |
Thanks! @bors r=@GuillaumeGomez rollup |
📌 Commit d39a1be has been approved by |
…c, r=GuillaumeGomez Check implementing type for `#[doc(hidden)]` Closes rust-lang#85526.
…c, r=GuillaumeGomez Check implementing type for `#[doc(hidden)]` Closes rust-lang#85526.
…askrgr Rollup of 8 pull requests Successful merges: - rust-lang#89950 (bootstrap: tweak verbosity settings) - rust-lang#89965 (Fix ICE with `let...else` and `ref mut`) - rust-lang#89974 (Nicer error message if the user attempts to do let...else if) - rust-lang#89987 (Check implementing type for `#[doc(hidden)]`) - rust-lang#89989 (rustdoc: Add static size assertion for `clean::Type`) - rust-lang#89990 (rustc_span: `Ident::invalid` -> `Ident::empty`) - rust-lang#89993 (Remove dead code from `compiletest::json`) - rust-lang#89996 (Bump backtrace) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
@camelid is there a reason you assigned Guillaume instead of a T-compiler reviewer? That seems pretty solidly in their wheelhouse to me ... missing_docs is not a rustdoc lint. |
I'm the one who implemented it, so I guess it's the reason? ;) |
My understanding was that we (T-rustdoc) are mainly the ones who maintain it, just like with the checks in |
Closes #85526.