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

Rename/alias "Run flycheck" command to something more obvious #14398

Open
ian-h-chamberlain opened this issue Mar 24, 2023 · 0 comments
Open

Comments

@ian-h-chamberlain
Copy link
Contributor

ian-h-chamberlain commented Mar 24, 2023

I agree that flycheck isn't the best fit, but check seems really generic so I am not sure if its apparent what check necessarily means for people when they read it for the first time either? I guess Run cargo check would be better? (shame we can't have the command label be dynamic).

Alternatively we can start pushing the flycheck term so that people know what it means. After adding this I was thinking of maybe changing the name of checkOnSave configs to check, flycheck, etc since it's no longer only tied to saving

Originally posted by @Veykril in #13785 (comment)

I found the above comment while trying to find the name of this command, which does not seem to be very discoverable with cargo check or rust check in the VSCode command palette:

Screenshot_20230324_093044

Screenshot_20230324_093247

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

No branches or pull requests

1 participant