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

doc: make impl block collapsible if it has an associated constant #71842

Merged
merged 1 commit into from
Jun 9, 2020

Conversation

tspiteri
Copy link
Contributor

@tspiteri tspiteri commented May 3, 2020

Fixes #71822.

@crlf0710
Copy link
Member

r? @kinnison

@crlf0710 crlf0710 added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 15, 2020
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.

This looks quite plausible to me, but I've not got a build-tree to hand to confirm it works right. Also because it's not generative I can't simply ask for a test case. @GuillaumeGomez should check it too.

@bors
Copy link
Contributor

bors commented May 27, 2020

☔ The latest upstream changes (presumably #72627) made this pull request unmergeable. Please resolve the merge conflicts.

@GuillaumeGomez
Copy link
Member

We can't check it for the moment, indeed. Can't wait to have the GUI tests merged...

Thanks @tspiteri !

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jun 8, 2020

📌 Commit da18df2 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 Jun 8, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 8, 2020
Rollup of 8 pull requests

Successful merges:

 - rust-lang#71842 (doc: make impl block collapsible if it has an associated constant)
 - rust-lang#72912 (Add new E0758 error code)
 - rust-lang#73008 (Update RELEASES.md)
 - rust-lang#73090 (Use `LocalDefId` directly in `Resolver::export_map`)
 - rust-lang#73118 (Improve the wording in documentation of std::mem::drop)
 - rust-lang#73124 (Removed lifetime parameters from Explanation of E0207 )
 - rust-lang#73138 (Use shorthand linker strip arguments in order to support MacOS)
 - rust-lang#73143 (Update books)

Failed merges:

r? @ghost
@bors bors merged commit 98bd22b into rust-lang:master Jun 9, 2020
@tspiteri tspiteri deleted the doc-impl-const branch June 9, 2020 08: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. 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: impl block with only associated constants not collapsible
6 participants