lsp-sonarlint icon indicating copy to clipboard operation
lsp-sonarlint copied to clipboard

C# support?

Open sollidsnake opened this issue 1 year ago • 2 comments

Are there plans to add C# support to this package?

sollidsnake avatar Aug 10 '23 20:08 sollidsnake

This package is based on the SonarLint language server from SonarLint for VSCode. As you can see, SonarLint for VSCode does not support C#, hence its language server also provides no support for C# analysis. In order to enable lsp-sonarlint to work with C# code, it first has to be supported in SonarLint for VSCode. As you can see in the SonarLint roadmap, it is not planned for the near future.

Feel free to suggest this feature using the "+ Submit idea" button in the SonarLint roadmap section.

necto avatar Aug 13 '23 19:08 necto

I see, thanks for the clarification. Hopefully it will happen soon now that the new Microsoft's LSP for C# is being released.

sollidsnake avatar Aug 14 '23 17:08 sollidsnake

I am not well versed in the C# ecosystem, but it looks like VSCode's sonarlint comes with omnisharp support now. I just submitted a PR to allow lsp-sonarlint to use the VSCode extension's analyzers, including omnisharp.

However a few additional tweaks may be needed to make omnisharp work. Feel free to try it out.

Horrih avatar Jun 08 '24 22:06 Horrih

https://github.com/emacs-lsp/lsp-sonarlint/pull/22 merged, so this should work :+1:

Sasanidas avatar Jun 28 '24 22:06 Sasanidas