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

Fix invalid debug display for associated consts #40587

Merged
merged 1 commit into from
Mar 20, 2017

Conversation

GuillaumeGomez
Copy link
Member

@GuillaumeGomez GuillaumeGomez commented Mar 16, 2017

Fixes #40568.

r? @rust-lang/docs

cc @SergioBenitez

@frewsxcv
Copy link
Member

@GuillaumeGomez Have a screenshot of the added test handy?

@GuillaumeGomez GuillaumeGomez force-pushed the rustdoc-const-display branch from 0ae1dc4 to 5364acb Compare March 16, 2017 23:27
@GuillaumeGomez
Copy link
Member Author

Sure:

screen shot 2017-03-17 at 00 28 21

@sophiajt
Copy link
Contributor

Out of curiosity, do we show both the struct and the fields today? I wasn't sure if that was something that was part of this PR.

@GuillaumeGomez
Copy link
Member Author

We do (take a look at the issue).

@sophiajt
Copy link
Contributor

@GuillaumeGomez

From looking here https://api.rocket.rs/rocket/http/struct.ContentType.html it seems like your screenshot doesn't reflect what it will look like. Do you have one that's closer to the original in terms of the amount of content?

@GuillaumeGomez
Copy link
Member Author

@jonathandturner: I have absolutely no idea how to run a specific binary from cargo doc (I can't document Rocket without it).

@frewsxcv
Copy link
Member

This seems fine to me. Unless @jonathandturner has any other questions here, r=me

@sophiajt
Copy link
Contributor

I'm good

@GuillaumeGomez
Copy link
Member Author

@bors: r=frewsxcv

@bors
Copy link
Contributor

bors commented Mar 19, 2017

📌 Commit 5364acb has been approved by frewsxcv

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Mar 20, 2017
…y, r=frewsxcv

Fix invalid debug display for associated consts

Fixes rust-lang#40568.

r? @rust-lang/docs

cc @SergioBenitez
bors added a commit that referenced this pull request Mar 20, 2017
Rollup of 9 pull requests

- Successful merges: #40241, #40281, #40398, #40521, #40532, #40554, #40566, #40581, #40587
- Failed merges:
@bors bors merged commit 5364acb into rust-lang:master Mar 20, 2017
@GuillaumeGomez GuillaumeGomez deleted the rustdoc-const-display branch March 20, 2017 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants