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

Only show notable traits if both types are the same #91366

Conversation

notriddle
Copy link
Contributor

Checking only their DefId doesn't work because all slices have the same
fake DefId.

Fixes #91347

Checking only their DefId doesn't work because all slices have the same
fake DefId.

Fixes rust-lang#91347
@rust-highfive
Copy link
Collaborator

r? @CraftSpider

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 29, 2021
@notriddle
Copy link
Contributor Author

r? @GuillaumeGomez

@GuillaumeGomez
Copy link
Member

Thanks!

r=me with nits fixed.

notriddle and others added 2 commits November 30, 2021 14:22
Co-authored-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
Co-authored-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
@GuillaumeGomez
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Nov 30, 2021

📌 Commit d4f71d8 has been approved by GuillaumeGomez

@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 Nov 30, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 1, 2021
…askrgr

Rollup of 4 pull requests

Successful merges:

 - rust-lang#91294 (Visit type in process_projection_elem.)
 - rust-lang#91340 (Bump compiler_builtins to 0.1.55 to bring in fixes for targets lackin…)
 - rust-lang#91366 (Only show notable traits if both types are the same)
 - rust-lang#91397 (Emit a warning on generic parameters with doc comments)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 2b8259a into rust-lang:master Dec 1, 2021
@rustbot rustbot added this to the 1.59.0 milestone Dec 1, 2021
@notriddle notriddle deleted the notriddle/only-show-notable-if-types-are-the-same branch December 1, 2021 14:33
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.

rustdoc: Read and Write show up as notable traits on &mut [u16]
6 participants