-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Diesel traits / methods are not resolved properly. #12589
Comments
Does https://github.com/oslac/ra-diesel-traits/blob/fa1b4e62ba5abf4df7847f2fe8c1cf545e9faea8/src/schema.rs#L1 get expanded or does rust-analyzer show an error for it? |
No errors from the |
Database connectors are tricky for rust-analyzer. #6833 (comment) |
Issue is still present on 2022-09-19 and nightly. Potentially related error log |
This works with the latest master branch of diesel, cause was their proc-macros using 2015 name resolution which r-a does not support currently |
@Veykril It's fine to close these issues, but you probably want to link them to an root issue for 2015 style name resolution, because that's an actual bug/missing feature in rust-analyzer. |
@QuadraticKid The specific instances that are reported in this issue are resolved. What you likely see is another bug that surfaced after working around the first one. The type inference breakes as soon as a @Veykril Should I fill new issues for the broken case or do you want to reopen the existing ones? |
New one sounds better (with crosslinking for context maybe) |
I submitted #14607 |
rust-analyzer version: 0.0.0 (427061da1 2022-06-19)
rustc 1.63.0-nightly (bb8c2f411 2022-06-19)
Issue: Rust-Analyzer fails to resolve some methods when using the Diesel ORM library. These would be the eq in
diesel::ExpressionMethods
and execute indiesel::RunQueryDsl
as far as I can tell.Example is available here: https://github.com/oslac/ra-diesel-traits
Expected: that RA properly resolves the traits / methods.
The text was updated successfully, but these errors were encountered: