Skip to content

Conversation

@Natural-selection1
Copy link
Contributor

close: #21016

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 11, 2025
@Natural-selection1 Natural-selection1 changed the title make visibility diagnostics for fields to correct location fix: make visibility diagnostics for fields to correct location Nov 11, 2025
Copy link
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

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

LGTM, but note that we have a situation (see #21008) that we can't merge PRs currently, so not merging this yet.

Thanks!

@Natural-selection1 Natural-selection1 force-pushed the visibility branch 2 times, most recently from e11bb3c to 64a0772 Compare November 17, 2025 13:36
@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Nov 18, 2025
@ChayimFriedman2 ChayimFriedman2 removed this pull request from the merge queue due to a manual request Nov 18, 2025
@ChayimFriedman2
Copy link
Contributor

What's going on?

@Natural-selection1
Copy link
Contributor Author

Natural-selection1 commented Nov 18, 2025

I don't know why the changes disappeared

Wait for me patiently for a few minutes : )

@Natural-selection1
Copy link
Contributor Author

I completely replicated the last commit before Marco Ieni's force-push

@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Nov 18, 2025
Merged via the queue into rust-lang:master with commit d0fbd88 Nov 18, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 18, 2025
@Natural-selection1 Natural-selection1 deleted the visibility branch November 18, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increase visibility assist doesn't play well with comments

4 participants