-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Checking an exercise takes too long #2071
Comments
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
I reported it in the Rust-Analyzer repo: rust-lang/rust-analyzer#17829 Until this is fixed, I will add a line print showing that Rustlings is running an exercise (not just frozen) and then make a release. |
This comment was marked as resolved.
This comment was marked as resolved.
Thanks you so much for handling this issue for me. I'll keep an eye on it to see if I can help with anything! |
rustlings
takes lots of time just to detect file changes
A message is now shown to indicate that an exercise is being checked so users know that they just need to wait. rustup update
cargo install rustlings The actual issue needs to be fixed in Rust-Analyzer now. |
I had the same issue on vscode and zed, checked zed's docs https://zed.dev/docs/languages/rust#large-projects-and-performance |
fix: run flycheck without rev_deps when target is specified Since querying for a crate's target is a call to salsa and therefore blocking, flycheck task is now deferred out of main thread by using `GlobalState`s `deferred_task_queue`. Fixes #17829 and rust-lang/rustlings#2071
…ykril fix: run flycheck without rev_deps when target is specified Since querying for a crate's target is a call to salsa and therefore blocking, flycheck task is now deferred out of main thread by using `GlobalState`s `deferred_task_queue`. Fixes rust-lang#17829 and rust-lang/rustlings#2071
This issue can be closed btw :) |
@alibektas I am afraid that this issue isn't fixed yet. You might have fixed one part of it, but Rust-Analyzer still seems to invalidate everything and check. Here is a video from a Windows VM with only 2 CPU cores. Screencast.from.2024-09-13.13-41-41.webmAt the beginning of the video, Rust-Analyzer is fully initialized and shows 168 errors and 26 warnings. Then, I press After it is done, I edit the file by adding I tested it with the stable Rust-Analyzer extension from 09.06. and the pre-release one from 09.12. This behavior is the same as in the video in the issue description. So I wonder whether rust-lang/rust-analyzer#17912 is applied yet and if it has any effect? |
The related issue is now rust-lang/rust-analyzer#18110 |
Description
This is the video that the issue happens:
2024-08-06.23-52-22.mp4
It can be shown that every time I update the code of an exercise, the
rust-analyzer
builds the wholeexercises
folder (it has like 156 exercises and the extension run the code check for every single of them). During this process, I cannot use any command (h
,n
,l
orq
). Only after it finishes, rustlings tells me if my solution is correct or not and directs me to the next exercise.I don't think it's a bug, but rather a code design issue. If I turn of
rust-analyzer
, it seems the issue's gone but I still have to wait for like 1-2 seconds forrustlings
to detect file changes.Please look into this issue and fix it. Thanks!
System Information
The text was updated successfully, but these errors were encountered: