-
Notifications
You must be signed in to change notification settings - Fork 42
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
Documentation on rust-analyzer seems outdated / anything to improve integration? #57
Comments
@bluenote10 I agree, that part of the FAQ appears to be outdated. Thanks for creating the issue! I have noticed similar issues occurring with the recent changes to Rust Analyzer, a lot of the issues appear to be related to rust-analyzer's macro expansion, so I ended up disabling them for the time being. I haven't noticed too many issues with rust analyzer though. It would be good to see what other folks using Rust Analyzer experience as well. |
Does anyone who uses Rust-Analyzer have time to investigate this issue?
You mean design the macros in a way that they don't fail hard but allow to handle the error? Maybe, but if |
@Bromeon I'm using Rust Analyzer, I'll doublecheck to see if there are any issues with a default install. It's been a while since I checked on my end. |
It should be noted that this big related tracking issue on rust-analyzer side rust-lang/rust-analyzer#11014 has been closed recently, so perhaps things are much better in the meantime (hope I can re-check as well soon). |
…ue with code completion.
I re-checked this issue with default settings from Rust Analyzer and it doesn't seem to be impacting anything on my end anymore. @bluenote10 please feel free to re-open this issue if you see any issues. |
@jacobsky I finally had some time to experiment with more recent rust-analyzer versions, and it looks like the issue with everything being underlined in red is indeed gone now. What still isn't working for me is if you have e.g. Do you get redirected to the actual implementation? |
Reopen until clarified. |
The section How do I get auto-completion with rust-analyzer? seems outdated, because
rust-analyzer.cargo.loadOutDirsFromCheck
no longer exists.In practice this is impacting my productivity quite a lot, because auto-completion within a native class exported function doesn't seem to work at all. Often, my IDE is in a state reporting simply "all broken"
... and if this happens when I'm having a weak moment it can take a while to spot the actual error.
To avoid getting impacted by this issue I have now started to implement more complex methods as free floating functions like
and forward calls from the methods, which is of course unnecessary boilerplate.
To narrow down the problem we probably need to answer the questions:
Is this just something broken with my setup, or is everyone using rust-analyzer facing these issues?
If it is setup related, what are the tricks to get it going, which we could update in the book?
If it turns out that it is fundamentally not working, the question may be if we can do something on library / macro side to improve it. Following the links posted by the rust-analyzer maintainer leads to a similar issue for
tokio-tracing
, caused by the usage ofsyn
and this proposal to solve it on their side. @Bromeon Is this proposal applicable to godot-rust as well?The text was updated successfully, but these errors were encountered: