Uh oh!
There was an error while loading. Please reload this page.
Hide default impls items - #54162
Conversation
rust-highfive
commented
Sep 12, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
bors
commented
Sep 12, 2018
☔ The latest upstream changes (presumably #53409) made this pull request unmergeable. Please resolve the merge conflicts. |
edd7fe2 to
f36fccaComparerust-highfive
commented
Sep 22, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
f36fcca to
11e8690Comparerust-highfive
commented
Sep 22, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
11e8690 to
25124ddCompareGuillaumeGomez
commented
Sep 22, 2018
Updated. |
rust-highfive
commented
Sep 22, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
GuillaumeGomez
commented
Sep 22, 2018
Not ready for the moment (because of #54492). |
XAMPPRocky
commented
Sep 26, 2018
Triage; @QuietMisdreavus Hello, are you able to review this PR? |
GuillaumeGomez
commented
Sep 26, 2018
@Aaronepower: Not ready yet so reviewing wouldn't be very useful for the moment... |
TimNN
commented
Oct 16, 2018
Ping from triage @GuillaumeGomez: what is the status of this PR? |
GuillaumeGomez
commented
Oct 17, 2018
Still need to be updated. Trying to do more urgent issues first. |
25124dd to
f729f4aCompareGuillaumeGomez
commented
Oct 21, 2018
Should be ready to go now! |
| pub fn is_associated(&self) -> bool { | ||
| match *self { | ||
| ItemEnum::TypedefItem(_, _) | | ||
| ItemEnum::AssociatedConstItem(_, _) | |
There was a problem hiding this comment.
I feel like associated consts don't need to be shown all the time, since we don't show const values any more.
TimNN
commented
Oct 30, 2018
Ping from triage @GuillaumeGomez: it looks like some updates have been requested to your PR. |
GuillaumeGomez
commented
Oct 30, 2018
I'll update it in the next days. |
QuietMisdreavus
left a comment
There was a problem hiding this comment.
r=me pending travis
rust-highfive
commented
Nov 2, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
QuietMisdreavus
commented
Nov 2, 2018
This is the second |
GuillaumeGomez
commented
Nov 2, 2018
I think the problem is a missing class here. |
cb5d220 to
ca65b43CompareUpdated the failing test accordingly (it was a missing class). |
QuietMisdreavus
commented
Nov 2, 2018
r=me pending travis |
GuillaumeGomez
commented
Nov 3, 2018
@bors: r=QuietMisdreavus |
bors
commented
Nov 3, 2018
📌 Commit ca65b43 has been approved by |
…tems, r=QuietMisdreavus Hide default impls items Follow up of rust-lang#51885. Fixesrust-lang#54025. cc @Mark-Simulacrum r? @QuietMisdreavus And screenshots of course: <img width="1440" alt="screen shot 2018-09-12 at 23 30 35" src="https://user-images.githubusercontent.com/3050060/45454424-1ff8d500-b6e4-11e8-9257-030322495d58.png"> <img width="1440" alt="screen shot 2018-09-12 at 23 30 42" src="https://user-images.githubusercontent.com/3050060/45454431-2424f280-b6e4-11e8-8d65-db0d85ac18f0.png">
Rollup of 5 pull requests Successful merges: - #54162 (Hide default impls items) - #55555 (Make `-Z ls` list the actual filename of external dependencies) - #55567 (add test for deriving Debug on uninhabited enum) - #55568 (test that rustdoc doesn't overflow on a big enum) - #55598 (publish-toolstate: ping maintainers when a tool builds again) Failed merges: r? @ghost
Follow up of #51885.
Fixes#54025.
cc @Mark-Simulacrum
r? @QuietMisdreavus
And screenshots of course: