Skip to content

feat: update references of tsserver to ts_ls#1131

Merged
feoh merged 1 commit into
nvim-lua:masterfrom
Cheveniko:master
Sep 10, 2024
Merged

feat: update references of tsserver to ts_ls#1131
feoh merged 1 commit into
nvim-lua:masterfrom
Cheveniko:master

Conversation

@Cheveniko

@Cheveniko Cheveniko commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

nvim-lspconfig renamed tsserver to ts_ls (see this PR)

This causes a warning everytime you start nvim because Mason hasn't changed tsserver name to ts_ls.

This PR fixes it by adding an if statement to check if the server name is tsserver and updates it to ts_ls in mason-lspconfig.setup (Credits)

Also it replaces every reference of tsserver to ts_ls

Comment thread init.lua
-- But for many setups, the LSP (`tsserver`) will work just fine
-- tsserver = {},
-- But for many setups, the LSP (`ts_ls`) will work just fine
-- ts_ls = {},

This comment was marked as outdated.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, I missed that 😵

It should remain tsserver in case anyone uncomments it

@shivambegin

Copy link
Copy Markdown

this is ready to merge as mason also renamed tsserver to ts_ls

@samarth-na

Copy link
Copy Markdown
Contributor

this should be added as soon as possible

@Cheveniko

Cheveniko commented Sep 9, 2024

Copy link
Copy Markdown
Contributor Author

Mason updated tsserver name to ts_ls (See this issue) so the fix isn't necessary anymore.

I've updated the commit to remove the fix and only update every reference of tsserver to ts_ls

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.

5 participants