-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Removes suggestions for misspelled private fields for structs defined in non-local crates #22516
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@leejunseok Looking good, thank you! Could you please also add a test for the new behaviour? It's not hard to do! You'll need to create an auxiliary // aux-build:struct-field-privacy.rs
extern crate struct-field-privacy;
use struct-field-privacy::A;
fn main () {
// ...
} struct-field-privacy.rs will then act as your external create. struct-field-hints.rs is an example of what you will need to put in in place of |
Er, it looks like the dashes in struct-field-privacy trips everything up. Is there another external crate I could use instead? |
Oops, just kidding, doing extern crate |
@leejunseok My bad, you'll need a license header for the test as well! With that r=me. :) |
Haha, thank you for your help. I'm relatively new to rust and I appreciate the friendly guidance! |
r? @jakub- |
@bors: r+ c87b722 |
@leejunseok No probs! Thanks again. :) |
⌛ Testing commit 9b67d07 with merge 9abf514... |
💔 Test failed - auto-linux-32-nopt-t |
The failure above was triggered by me; this PR is part of the rollup so it's redundant to test it individually (and given the state of the queue, rather wasteful). If the rollup doesn't go through in a couple of tries, ask for a retry on this 😄 Sorry for the trouble! |
closes #22421