Skip to content
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

Don't suggest inaccessible fields #93039

Merged
merged 1 commit into from
Jan 31, 2022
Merged

Conversation

terrarier2111
Copy link
Contributor

Fixes: #92999

@rust-highfive
Copy link
Collaborator

r? @nagisa

(rust-highfive has picked a reviewer for you, use r? to override)

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jan 18, 2022
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 18, 2022
@rust-log-analyzer

This comment has been minimized.

@terrarier2111 terrarier2111 force-pushed the fix-field-help branch 2 times, most recently from c9b8125 to 0c02f71 Compare January 19, 2022 16:24
@nagisa
Copy link
Member

nagisa commented Jan 22, 2022

I think the commit message/PR description could be improved to be more descriptive of change being made here, before this is merged. Once that's done and the nit is fixed (or there's actually a good reason to do the things the way done right now) r=me.

@terrarier2111 terrarier2111 changed the title Fix field help Don't suggest inaccessible fields Jan 26, 2022
@terrarier2111
Copy link
Contributor Author

I think the commit message/PR description could be improved to be more descriptive of change being made here, before this is merged. Once that's done and the nit is fixed (or there's actually a good reason to do the things the way done right now) r=me.

Done.

@rust-log-analyzer

This comment has been minimized.

@nagisa
Copy link
Member

nagisa commented Jan 30, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Jan 30, 2022

📌 Commit ecd06e1 has been approved by nagisa

@bors bors 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-review Status: Awaiting review from the assignee but also interested parties. labels Jan 30, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jan 30, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 31, 2022
…askrgr

Rollup of 8 pull requests

Successful merges:

 - rust-lang#90277 (Improve terminology around "after typeck")
 - rust-lang#92918 (Allow eliding GATs in expression position)
 - rust-lang#93039 (Don't suggest inaccessible fields)
 - rust-lang#93155 (Switch pretty printer to block-based indentation)
 - rust-lang#93214 (Respect doc(hidden) when suggesting available fields)
 - rust-lang#93347 (Make `char::DecodeUtf16::size_hist` more precise)
 - rust-lang#93392 (Clarify documentation on char::MAX)
 - rust-lang#93444 (Fix some CSS warnings and errors from VS Code)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 71efe90 into rust-lang:master Jan 31, 2022
@rustbot rustbot added this to the 1.60.0 milestone Jan 31, 2022
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.

Incorrect suggestion of private struct members
7 participants