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

Re-exporting std traits confuses Rust Analyzer #351

Open
TheLostLambda opened this issue Apr 23, 2024 · 4 comments
Open

Re-exporting std traits confuses Rust Analyzer #351

TheLostLambda opened this issue Apr 23, 2024 · 4 comments
Labels
Milestone

Comments

@TheLostLambda
Copy link
Contributor

I'm not sure if this is just a configuration issue on my end, but rust-analyzer seems convinced that the only version of traits like Display or Into come from this library.

This has been particularly annoying when I've gone to import one of these traits automatically, but it only suggests importing the derive_more version. I'd rather not have imports from derive_more in the modules I'm not using it, so I've been having to manually import things every time this clash happens.

A couple of examples — first from a diagnostic:
image
It should be noted that the file that error is coming from doesn't mention or import derive_more anywhere. It's just a carry-over from other modules in the crate using it.

And the annoying no-correct-import case:
image

I'm not sure if there is a clever way around this, but if not, I'd honestly prefer to have to separately import the trait from std!

What do others think / what was the rationale for re-exporting those std traits to begin with?

@TheLostLambda
Copy link
Contributor Author

Oh, importantly this is the 1.0.0-beta.6 — I think this is a change introduced by 1.0.0?

@JelteF
Copy link
Owner

JelteF commented Apr 23, 2024

That does seem like quite a bad side-effect of the change. @tyranron what do you think about this. Can you think of a reason to address this? I remember some colleagues complaining about this same issue.

@JelteF JelteF added this to the 1.0.0 milestone Apr 23, 2024
@JelteF JelteF added the bug label Apr 23, 2024
@JelteF
Copy link
Owner

JelteF commented Apr 23, 2024

This sounds related to this rust-analyzer issue. rust-lang/rust-analyzer#11698

We do mark these reexported traits as doc(hidden). So probably the best is to fix this rust-analyzer issue.

@tyranron
Copy link
Collaborator

@JelteF yup, I agree that this should be rather fixed on the rust-analyzer side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants