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

rustdoc: Hide struct and enum variant constructor imports #52196

Merged
merged 1 commit into from
Jul 10, 2018

Conversation

ollie27
Copy link
Member

@ollie27 ollie27 commented Jul 9, 2018

This is fallout from #51425. The duplicate variant imports can be seen here for example.

This is fixing a regression so could be backported to beta.

r? @QuietMisdreavus

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 9, 2018
@QuietMisdreavus
Copy link
Member

Whoops, i thought i had gotten it with that change you removed here, but i guess not. Thanks for the fix! r=me pending travis.

re: backport, i don't think this is severe enough to warrant a backport, but it's worth seeing what the rest of @rust-lang/rustdoc thinks.

@GuillaumeGomez
Copy link
Member

I'd be in favor for a backport.

@QuietMisdreavus QuietMisdreavus added beta-nominated Nominated for backporting to the compiler in the beta channel. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Jul 10, 2018
@QuietMisdreavus
Copy link
Member

@bors r+

@steveklabnik Care to be the tiebreaking vote on whether to backport?

@bors
Copy link
Contributor

bors commented Jul 10, 2018

📌 Commit 6b1d584 has been approved by QuietMisdreavus

@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 Jul 10, 2018
@bors
Copy link
Contributor

bors commented Jul 10, 2018

⌛ Testing commit 6b1d584 with merge 90bd83c...

bors added a commit that referenced this pull request Jul 10, 2018
rustdoc: Hide struct and enum variant constructor imports

This is fallout from #51425. The duplicate variant imports can be seen [here](https://doc.rust-lang.org/nightly/std/prelude/v1/index.html) for example.

This is fixing a regression so could be backported to beta.

r? @QuietMisdreavus
@bors
Copy link
Contributor

bors commented Jul 10, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: QuietMisdreavus
Pushing 90bd83c to master...

@bors bors merged commit 6b1d584 into rust-lang:master Jul 10, 2018
@ollie27 ollie27 deleted the rustdoc_ctor_imports branch July 12, 2018 15:04
@pietroalbini
Copy link
Member

Ping @rust-lang/rustdoc, should this be backported to beta?

@GuillaumeGomez GuillaumeGomez added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jul 12, 2018
@GuillaumeGomez
Copy link
Member

Yes it should.

@pietroalbini pietroalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jul 12, 2018
bors added a commit that referenced this pull request Jul 12, 2018
[beta] Rollup backports

Merged and approved:

* #51722: Updated RELEASES for 1.28.0
* #52193: step_by: leave time of item skip unspecified
* #52194: Remove rustdoc's plugins feature
* #52196: rustdoc: Hide struct and enum variant constructor imports
* #52310: Backport 1.27.1 release notes to master

r? @ghost
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. 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.

6 participants