Skip to content

Match expressions had not been updated to also show the match_source,…#157769

Open
AlexCeleste wants to merge 1 commit into
rust-lang:mainfrom
AlexCeleste:thir-missing-match-source
Open

Match expressions had not been updated to also show the match_source,…#157769
AlexCeleste wants to merge 1 commit into
rust-lang:mainfrom
AlexCeleste:thir-missing-match-source

Conversation

@AlexCeleste

Copy link
Copy Markdown
Contributor

… since the time that was added

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 11, 2026
@rustbot

rustbot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

r? @petrochenkov

rustbot has assigned @petrochenkov.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 73 candidates
  • Random selection from 19 candidates

@rust-log-analyzer

This comment has been minimized.

@petrochenkov

Copy link
Copy Markdown
Contributor

r? @Nadrieril maybe, I'm not sure what this is about

@rustbot rustbot assigned Nadrieril and unassigned petrochenkov Jun 11, 2026
@rustbot

rustbot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Nadrieril is currently at their maximum review capacity.
They may take a while to respond.

@AlexCeleste AlexCeleste force-pushed the thir-missing-match-source branch from 406d85a to ac251f4 Compare June 12, 2026 09:51
@rustbot

rustbot commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main 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.

@rust-log-analyzer

This comment has been minimized.

@AlexCeleste AlexCeleste force-pushed the thir-missing-match-source branch from ac251f4 to 4da45bc Compare June 12, 2026 11:39
@rust-log-analyzer

This comment has been minimized.

… since the time that was added

update existing match tests as well

remove hashes from output

remove specific DefIds for core from output
@AlexCeleste AlexCeleste force-pushed the thir-missing-match-source branch from 4da45bc to 481db97 Compare June 12, 2026 12:27
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants