Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

feat: 🎸 add rust fly check command #379

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

IWANABETHATGUY
Copy link

@IWANABETHATGUY IWANABETHATGUY commented May 5, 2023

  1. rust cargo check after each save is pretty costly, especially when the project became very large.
  2. flyCheck is very useful in such a scenario, after running this command rust analyzer will on demand trigger cargo check and push the diagnostics into neovim lsp client

@IWANABETHATGUY
Copy link
Author

20230505-140239.mp4

@IWANABETHATGUY
Copy link
Author

related code in rust-analyzer vscode extension https://github.com/rust-lang/rust-analyzer/blob/master/editors/code/src/lsp_ext.ts#L45-L47

@pgherveou
Copy link

Looks pretty cool, do you need to disable checkOnSave when you want to use this?

@IWANABETHATGUY
Copy link
Author

In theory, yes. Because if you enable checkOnSave, save will trigger a fly check already. This is only useful when cargo check cost pretty long time.

@MunifTanjim
Copy link

MunifTanjim pushed a commit to MunifTanjim/rust-tools.nvim that referenced this pull request May 20, 2023
@pgherveou
Copy link

In theory, yes. Because if you enable checkOnSave, save will trigger a fly check already. This is only useful when cargo check cost pretty long time.

So basically you only need to make sure you have this checkOnSave turned off to benefit from that feature.
Might be worth adding a note about it in the doc so that people knows about it

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

Successfully merging this pull request may close these issues.

3 participants