Skip to content

Fixes primitive sidebar link generation #55682

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 2 commits into from
Dec 4, 2018

Conversation

GuillaumeGomez
Copy link
Member

Fixes #50746.
Fixes #55656.

r? @QuietMisdreavus

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 5, 2018
@jkordish
Copy link

Ping from triage: @QuietMisdreavus have you had time to review this PR?

@QuietMisdreavus
Copy link
Member

Can you add a test? You can cause the linked issue by giving a struct two functions with the same name:

pub struct SomeStruct<T> { _inner: T }

impl SomeStruct<()> {
    pub fn some_fn(&self) {}
}

impl SomeStruct<usize> {
    pub fn some_fn(&self) {}
}

@GuillaumeGomez
Copy link
Member Author

Ok!

@GuillaumeGomez GuillaumeGomez force-pushed the primitive-sidebar-link-gen branch from 6720870 to c746ecf Compare November 21, 2018 22:22
@GuillaumeGomez
Copy link
Member Author

Added the test.

@TimNN
Copy link
Contributor

TimNN commented Nov 27, 2018

Ping from triage @QuietMisdreavus: This PR requires your review.

@QuietMisdreavus
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 3, 2018

📌 Commit c746ecf 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 Dec 3, 2018
@QuietMisdreavus
Copy link
Member

(Dunno how i closed the PR, but that wasn't supposed to happen...)

@bors
Copy link
Collaborator

bors commented Dec 4, 2018

⌛ Testing commit c746ecf with merge 91d5d56...

bors added a commit that referenced this pull request Dec 4, 2018
…uietMisdreavus

Fixes primitive sidebar link generation

Fixes #50746.
Fixes #55656.

r? @QuietMisdreavus
@bors
Copy link
Collaborator

bors commented Dec 4, 2018

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

@bors bors merged commit c746ecf into rust-lang:master Dec 4, 2018
@GuillaumeGomez GuillaumeGomez deleted the primitive-sidebar-link-gen branch December 4, 2018 08:48
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