-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Conversation
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. |
I'd be in favor for a backport. |
@bors r+ @steveklabnik Care to be the tiebreaking vote on whether to backport? |
📌 Commit 6b1d584 has been approved by |
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
☀️ Test successful - status-appveyor, status-travis |
Ping @rust-lang/rustdoc, should this be backported to beta? |
Yes it should. |
[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
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