docs(lsp): fix the way of removing items from skipped_servers

[mende1]

Description

The currently way to remove items from skipped_servers list doesn't make any effect on the original list lvim.lsp.automatic_configuration.skipped_servers:

vim.tbl_map(function(server)
  return server ~= "emmet_ls"
end, lvim.lsp.automatic_configuration.skipped_servers)

The function vim.tbl_map only returns a new list with the certain function applied, and doesn't modify the original one. Also, by the example return server ~= "emmet_ls", it doesn't return the list with filtered values, and yes a list with booleans resulted by the function.

The solution for this is just assigning the returned list from the vim.tbl_map function inside the original list lvim.lsp.automatic_configuration.skipped_servers and also changing the function to vim.tbl_filter, because this way makes sense with the function used in the example return server ~= "emmet_ls" to recieve a list with filtered values.

lvim.lsp.automatic_configuration.skipped_servers = vim.tbl_filter(function(server)
  return server ~= "emmet_ls"
end, lvim.lsp.automatic_configuration.skipped_servers)

Fixes #2501

[kylo252]

related #2503

[lvimuser]

@kylo252 This should be fine? fwiw, that's what I do and been recommending over discord. This resolves the issue without doing anything special, since it's just a lua table.

@mende1 Can you squash it and fix the commit linter warning?

Error: You have commit messages with errors
✖   body's lines must not be longer than 100 characters [body-max-line-length]

[kylo252]

Thanks!