Skip to content

Fix: Update the 'cmd' setting for the 'jsonls' LSP server #3822

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

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

Conversation

austindd
Copy link

@austindd austindd commented May 6, 2025

I encountered a bug with the default jsonls LSP server config. Perhaps I was missing something, but this change fixed my issue.

Before, the default cmd for jsonls was this:

    cmd = { 'vscode-json-language-server', '--stdio' },

This caused a bug where the LSP server was not loaded. I changed it to this (removed a dash):

    cmd = { 'vscode-json-languageserver', '--stdio' },

This change fixed the problem and enabled the LSP server to be attached.

@austindd austindd requested a review from glepnir as a code owner May 6, 2025 23:05
@austindd austindd changed the title Fix the 'cmd' setting for the 'jsonls' LSP server Fix: Update the 'cmd' setting for the 'jsonls' LSP server May 6, 2025
@justinmk
Copy link
Member

justinmk commented May 7, 2025

ok, but that doesn't match the docs (in the very same file).

@hrsh7th does this look right ?

@lithammer
Copy link
Collaborator

How did you install it (os, platform, package manager etc)?

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

Successfully merging this pull request may close these issues.

3 participants