-
Notifications
You must be signed in to change notification settings - Fork 1.9k
fix: make visibility diagnostics for fields to correct location #21018
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: make visibility diagnostics for fields to correct location #21018
Conversation
5a1df07 to
52df627
Compare
52df627 to
1700dd3
Compare
ChayimFriedman2
left a comment
There was a problem hiding this 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!
e11bb3c to
64a0772
Compare
64a0772 to
6654b4e
Compare
|
What's going on? |
|
I don't know why the changes disappeared Wait for me patiently for a few minutes : ) |
6654b4e to
cc66e1d
Compare
|
I completely replicated the last commit before Marco Ieni's force-push |
close: #21016