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.

Sign up for GitHub

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

Jump to bottom

Change default language server for 'v' from 'vls' to 'v ls' #5677

Merged
merged 2 commits into from
Jan 25, 2023
Merged

Change default language server for 'v' from 'vls' to 'v ls' #5677

merged 2 commits into from
Jan 25, 2023

Conversation

antoniusnaumann
Copy link
Contributor

According to the V Language Server Repository README, the recommended way to install the V Language Server is via the v CLI. Therefore, the default config should expect it to be called by using v ls not vls.

@the-mikedavis the-mikedavis added S-waiting-on-review Status: Awaiting review from a maintainer. A-language-support Area: Support for programming/text languages labels Jan 25, 2023
@the-mikedavis
Copy link
Member

Could you run cargo xtask docgen and commit the results? That will clear up the CI failure

@antoniusnaumann
Copy link
Contributor Author

Done!

@the-mikedavis the-mikedavis merged commit 56c0810 into helix-editor:master Jan 25, 2023
@the-mikedavis the-mikedavis mentioned this pull request Mar 12, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@the-mikedavis the-mikedavis the-mikedavis approved these changes

Assignees
No one assigned
Labels
A-language-support Area: Support for programming/text languages S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@antoniusnaumann @the-mikedavis