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

In '--health <lang>', print expected cmd when missing in $PATH #5193

Closed
helmesjo opened this issue Dec 17, 2022 · 0 comments · Fixed by #5195
Closed

In '--health <lang>', print expected cmd when missing in $PATH #5193

helmesjo opened this issue Dec 17, 2022 · 0 comments · Fixed by #5195
Labels
C-enhancement Category: Improvements E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors

Comments

@helmesjo
Copy link

helmesjo commented Dec 17, 2022

I tried looking through the issues but couldn't find any mention, but basically a small but useful enhancement would be to actually list what executable/cmd helix expects when it says "Not found in $PATH".

So here:

$ hx --health cpp
Configured language server: clangd
Binary for language server: /usr/bin/clangd
Configured debug adapter: lldb-vscode
Binary for debug adapter: Not found in $PATH
Highlight queries: ✓
Textobject queries: ✓
Indent queries: ✓

It would be very neat if it instead said:
Binary for debug adapter: 'lldb-vscode' not found in $PATH

By instead doing the full hx --health one can figure it out, but I guess there is no reason it couldn't be mentioned in both.

@helmesjo helmesjo added the C-enhancement Category: Improvements label Dec 17, 2022
@dead10ck dead10ck added E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors labels Dec 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: Improvements E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants