Skip to content

Hide cross-crate #[doc(hidden)] associated items in trait impls #95769

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

Merged
merged 1 commit into from
Apr 9, 2022

Conversation

fmease
Copy link
Member

@fmease fmease commented Apr 7, 2022

Fixes #95717.

r? @GuillaumeGomez
This is the bug I ran into in #95316.

@rustbot label T-rustdoc A-cross-crate-reexports

@rustbot rustbot added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Apr 7, 2022
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 7, 2022
@rustbot rustbot added the A-cross-crate-reexports Area: Documentation that has been re-exported from a different crate label Apr 7, 2022
@GuillaumeGomez
Copy link
Member

Other than the test, it looks good to me, thanks!

cc @camelid to double check the code changes.

@fmease fmease force-pushed the fix-issue-95717 branch 2 times, most recently from bbe3da1 to 80be793 Compare April 7, 2022 18:40
@fmease fmease force-pushed the fix-issue-95717 branch from 80be793 to 4623d51 Compare April 7, 2022 18:40
@camelid
Copy link
Member

camelid commented Apr 7, 2022

@GuillaumeGomez Sorry, I don't have time to review this, but nothing stood out when I glanced at the diff. I'll leave it to you :)

@GuillaumeGomez
Copy link
Member

Thanks for taking a look in any case! :)

@fmease: Thanks for the PR and very good idea to use @count!

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 8, 2022

📌 Commit 4623d51 has been approved by GuillaumeGomez

@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 Apr 8, 2022
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Apr 8, 2022
…Gomez

Hide cross-crate `#[doc(hidden)]` associated items in trait impls

Fixes rust-lang#95717.

r? `@GuillaumeGomez`
This is the bug I ran into in rust-lang#95316.

`@rustbot` label T-rustdoc A-cross-crate-reexports
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Apr 9, 2022
…Gomez

Hide cross-crate `#[doc(hidden)]` associated items in trait impls

Fixes rust-lang#95717.

r? ``@GuillaumeGomez``
This is the bug I ran into in rust-lang#95316.

``@rustbot`` label T-rustdoc A-cross-crate-reexports
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 9, 2022
Rollup of 6 pull requests

Successful merges:

 - rust-lang#95308 (Reduce the amount of unstable features used in libproc_macro)
 - rust-lang#95676 (Update RLS)
 - rust-lang#95769 (Hide cross-crate `#[doc(hidden)]` associated items in trait impls)
 - rust-lang#95785 (interpret: err instead of ICE on size mismatches in to_bits_or_ptr_internal)
 - rust-lang#95802 (fix unused constant warning on some Windows targets)
 - rust-lang#95810 (Use `format-args-capture` and remove unnecessary nested blocks)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 24fa80d into rust-lang:master Apr 9, 2022
@rustbot rustbot added this to the 1.62.0 milestone Apr 9, 2022
@fmease fmease deleted the fix-issue-95717 branch April 12, 2022 01:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-cross-crate-reexports Area: Documentation that has been re-exported from a different crate S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rustdoc: #[doc(hidden)] associated items of traits re-exported from external crates are not hidden in certain impl blocks
6 participants