Skip to content

When issuing suggestions for missing trait items, label unstable items - #160250

Open
joshtriplett wants to merge 2 commits into
rust-lang:mainfrom
joshtriplett:must-implement-one-of-no-unstable-suggestions
Open

When issuing suggestions for missing trait items, label unstable items#160250
joshtriplett wants to merge 2 commits into
rust-lang:mainfrom
joshtriplett:must-implement-one-of-no-unstable-suggestions

Conversation

@joshtriplett

Copy link
Copy Markdown
Member

When we suggest implementing a missing trait item (or group of items from must_implement_one_of), label any unstable items, so the user doesn't get confused by suggestions to implement an unstable item parallel to suggestions to implement stable items.

@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 Jul 31, 2026
@rustbot

rustbot commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

r? @JohnTitor

rustbot has assigned @JohnTitor.
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, types
  • compiler, types expanded to 75 candidates
  • Random selection from 18 candidates

@joshtriplett

Copy link
Copy Markdown
Member Author

Followup for #160106 .

Unblocks #106643 .

Comment thread compiler/rustc_hir_analysis/src/diagnostics.rs Outdated

@JohnTitor JohnTitor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, r=me with fixing the typo above

View changes since this review

@JohnTitor JohnTitor 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 Jul 31, 2026
When we suggest implementing a missing trait item (or group of items
from `must_implement_one_of`), label any unstable items, so the user
doesn't get confused by suggestions to implement an unstable item
parallel to suggestions to implement stable items.
Includes a test for unstable trait methods.
@joshtriplett
joshtriplett force-pushed the must-implement-one-of-no-unstable-suggestions branch from 29d1888 to c4058be Compare August 1, 2026 01:33
@joshtriplett

Copy link
Copy Markdown
Member Author

@bors r=JohnTitor rollup

@rust-bors

rust-bors Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

📌 Commit c4058be has been approved by JohnTitor

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 Aug 1, 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-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.

3 participants