Skip to content

Trait implementations order random #117391

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
bend-n opened this issue Oct 30, 2023 · 1 comment · Fixed by #117521
Closed

Trait implementations order random #117391

bend-n opened this issue Oct 30, 2023 · 1 comment · Fixed by #117521
Assignees
@GuillaumeGomez
Labels
A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools A-rustdoc-js Area: Rustdoc's JS front-end T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@bend-n
Copy link
Contributor

bend-n commented Oct 30, 2023

Location

foreign-impls

Summary

Displeasing order,
image
Expect sorted, or source order.

@GuillaumeGomez
@bend-n bend-n added the A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools label Oct 30, 2023
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Oct 30, 2023
@GuillaumeGomez GuillaumeGomez added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. A-rustdoc-js Area: Rustdoc's JS front-end and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Oct 30, 2023
@GuillaumeGomez
Copy link
Member

I'll take a look in the next days.

@GuillaumeGomez GuillaumeGomez mentioned this issue Nov 2, 2023
Merged
@bors bors closed this as completed in b20f40d Nov 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@GuillaumeGomez GuillaumeGomez

Labels
A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools A-rustdoc-js Area: Rustdoc's JS front-end T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix order of implementations in the "implementations on foreign types" section GuillaumeGomez/rust
3 participants
@GuillaumeGomez @rustbot @bend-n