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

Diagnostics are not displayed anymore in Cargoless setup #18847

Closed
bobozaur opened this issue Jan 6, 2025 · 5 comments · Fixed by #18848
Closed

Diagnostics are not displayed anymore in Cargoless setup #18847

bobozaur opened this issue Jan 6, 2025 · 5 comments · Fixed by #18848
Labels
C-bug Category: bug

Comments

@bobozaur
Copy link
Contributor

bobozaur commented Jan 6, 2025

rust-analyzer version: (eg. output of "rust-analyzer: Show RA Version" command, accessible in VSCode via Ctrl/⌘+Shift+P): v0.3.2237

rustc version: (eg. output of rustc -V): nightly/2024-08-23

editor or extension: (eg. VSCode, Vim, Emacs, etc. For VSCode users, specify your extension version; for users of other editors, provide the distribution if applicable): Zed 0.167.1 and VSCode 1.96.2

relevant settings: (eg. client settings, or environment variables like CARGO, RUSTC, RUSTUP_HOME or CARGO_HOME): Cargoless setup, using Bazel instead.

repository link (if public, optional): (eg. rust-analyzer): N/A

description:
Diagnostics are cleared immediately after being generated and displayed (they get cleared so fast that most of the time it's like they were not displayed at all). I tracked the issue down to this commit but I can't tell what's wrong with it. Note that the issue does not occur in a Cargo workspace.

@bobozaur bobozaur added the C-bug Category: bug label Jan 6, 2025
@bobozaur bobozaur changed the title Diagnostics are not displayed in Cargoless setup Diagnostics are not displayed anymore in Cargoless setup Jan 6, 2025
@Veykril
Copy link
Member

Veykril commented Jan 6, 2025

The upcoming release should have this fixed (I'll leave this open though until you can confirm). The release is a bit delayed today as we want to fix a pretty bad bug still.

@bobozaur
Copy link
Contributor Author

bobozaur commented Jan 6, 2025

@Veykril I tried the latest master but I still have the issue. Is there a specific commit I could use to build and test locally?

@Veykril
Copy link
Member

Veykril commented Jan 6, 2025

Ah apologies I missed the

Note that the issue does not occur in a Cargo workspace.

@Veykril
Copy link
Member

Veykril commented Jan 6, 2025

Oh, I see the issue, silly mistake :)

@bobozaur
Copy link
Contributor Author

bobozaur commented Jan 6, 2025

@Veykril Thanks for taking care of this so quickly! Can confirm it works.

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

Successfully merging a pull request may close this issue.

2 participants