Skip to content

rustdoc: Add missing src links for generic impls on trait pages #47039

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

Merged
merged 1 commit into from
Jan 7, 2018

Conversation

ollie27
Copy link
Member

@ollie27 ollie27 commented Dec 27, 2017

implementor2item would return None for generic impls so instead this clones the entire clean::Item into the implementors map which simplifies some code.

`implementor2item` would return `None` for generic impls so instead this clones the entire `clean::Item` into the `implementors` map which simplifies some code.
@rust-highfive
Copy link
Contributor

r? @steveklabnik

(rust_highfive has picked a reviewer for you, use r? to override)

@kennytm
Copy link
Member

kennytm commented Jan 3, 2018

Thanks for the PR! We’ll periodically check in on it to make sure that @steveklabnik or someone else from the team reviews it soon.

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 3, 2018
@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+

@bors
Copy link
Collaborator

bors commented Jan 7, 2018

📌 Commit 3d11d20 has been approved by GuillaumeGomez

@bors
Copy link
Collaborator

bors commented Jan 7, 2018

⌛ Testing commit 3d11d20 with merge 8724337...

bors added a commit that referenced this pull request Jan 7, 2018
rustdoc: Add missing src links for generic impls on trait pages

`implementor2item` would return `None` for generic impls so instead this clones the entire `clean::Item` into the `implementors` map which simplifies some code.
@kennytm kennytm 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 Jan 7, 2018
@bors
Copy link
Collaborator

bors commented Jan 7, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: GuillaumeGomez
Pushing 8724337 to master...

@bors bors merged commit 3d11d20 into rust-lang:master Jan 7, 2018
@ollie27 ollie27 deleted the rustdoc_trait_impl_src branch January 7, 2018 17:19
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants