Skip to content

Conversation

ada4a
Copy link
Contributor

@ada4a ada4a commented Aug 20, 2025

this is based #15519, but mainly to avoid gnarly rebase conflicts later

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Aug 20, 2025

r? @Alexendoo

rustbot has assigned @Alexendoo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 20, 2025
@ada4a
Copy link
Contributor Author

ada4a commented Aug 20, 2025

r? clippy

@rustbot rustbot assigned blyxyas and unassigned Alexendoo Aug 20, 2025
@ada4a ada4a force-pushed the non_canonical_impls branch 2 times, most recently from c0dd48b to e10a7a2 Compare August 22, 2025 13:32
@rustbot

This comment has been minimized.

ada4a added 3 commits August 24, 2025 08:14
too many lines no more!
imo it's a bit more clear than checking for the same thing (`expr.kind`)
in multiple branches
@ada4a ada4a force-pushed the non_canonical_impls branch from e10a7a2 to e93a9fa Compare August 24, 2025 06:16
@rustbot
Copy link
Collaborator

rustbot commented Aug 24, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@blyxyas
Copy link
Member

blyxyas commented Aug 24, 2025

:0 triagebot was updated with new functionality. Neat!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants