Skip to content

Make all multipart suggestions verbose#152596

Open
estebank wants to merge 1 commit intorust-lang:mainfrom
estebank:multipart_suggestions
Open

Make all multipart suggestions verbose#152596
estebank wants to merge 1 commit intorust-lang:mainfrom
estebank:multipart_suggestions

Conversation

@estebank
Copy link
Contributor

The ShowAlways style of suggestions is usually easier to understand than the inline style.

@rustbot
Copy link
Collaborator

rustbot commented Feb 14, 2026

Some changes occurred in src/tools/clippy

cc @rust-lang/clippy

Some changes occurred in match checking

cc @Nadrieril

HIR ty lowering was modified

cc @fmease

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-clippy Relevant to the Clippy team. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 14, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 14, 2026

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 68 candidates
  • Random selection from 17 candidates

@rust-log-analyzer

This comment has been minimized.

@rust-bors

This comment has been minimized.

@petrochenkov
Copy link
Contributor

This is good, I never understood the logic behind choosing between multipart_suggestion_verbose and multipart_suggestion, perhaps there was none.

r=me after rebasing and making CI green.
@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 16, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 16, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@estebank
Copy link
Contributor Author

This is good, I never understood the logic behind choosing between multipart_suggestion_verbose and multipart_suggestion, perhaps there was none.

Honestly, inertia. The initial suggestion rendering was inline. Much later, I introduced the verbose mode for specific cases. Since then, the verbose mode became much better, to the point where it makes sense for it to be the default in most cases. There's a handful of cases where the inline suggestions make more sense (when it is a single word being changed, mainly). The multipart suggestion was introduced when inline was the better mode. This switch makes sense to me given how much easier to read the patch format is (specially because it makes incorrect suggestion spans more evident!).

The ShowAlways style of suggestions is usually easier to understand than the inline style.
@estebank estebank force-pushed the multipart_suggestions branch from c6ad919 to 37684bd Compare February 18, 2026 18:33
@rustbot
Copy link
Collaborator

rustbot commented Feb 18, 2026

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.

@estebank
Copy link
Contributor Author

@bors r=petrochenkov

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 18, 2026

📌 Commit 37684bd has been approved by petrochenkov

It is now in the queue for this repository.

@rust-bors rust-bors bot 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Feb 18, 2026
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. T-clippy Relevant to the Clippy team. 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.

4 participants

Comments