Fix JsonNull conversion error #402
Conversation
Some LSP clients do not send values outside of of configuration
namespace. For example Neovim does not support sending client
configurations to mimic the VScode behavior. Following in Lua is not
possible.
```lua
require('sonarlint').setup({
server = {
cmd = {
-- …
},
settings = {
sonarlint = {
-- …
},
["files.exclude"] = { ["**/.git"] = true },
}
},
})
```
This commit makes sure that sonarlint language server still functions if
the values won't be sent. Otherwise, the language server errors while
loading the project.
|
The error has been reported here: https://gitlab.com/schrieveslaach/sonarlint.nvim/-/issues/24 |
|
Hey @sophio-japharidze-sonarsource could you please check this PR when you can? Thank you for your understanding. |
|
Hey @schrieveslaach, thank you for this new contribution! I copied it as #410 to let our CI checks run on it. If this is OK for you, I'll make sure that this other PR is reviewed and merged for the next release. |
|
@jblievremont, yeah sure. Just tag me if you need me to do something. |
|
Any news on this? Would be great to get SonarLint working again in neovim. |
|
Merged as part of #410, the changes should be available in the next public release of the language server. Thanks again! |
|
@jblievremont, thanks for driving this fix forward. |
|
Hello 👋🏻 FYI sonarlint-language-server 3.13.0.75723 has just been released and should be available on Maven Central on the next cache refresh. |
Nice. For use with Mason we still have to wait for the VSCode plugin to be released though. |
Some LSP clients do not send values outside of of configuration namespace. For example Neovim does not support sending client configurations to mimic the VScode behavior. Following in Lua is not possible.
This commit makes sure that sonarlint language server still functions if the values won't be sent. Otherwise, the language server errors while loading the project.
Part of