Skip to content
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

Refactor lsp-sonarlint and cfamily support #20

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

Conversation

xeonxu
Copy link

@xeonxu xeonxu commented Sep 21, 2023

To use AIO solution of VS Code's sonarlint plugin. Optimize to use activation-fn instead of major-mode method when creating lsp-client. This will let you changing support languages in runtime.

And big update, lsp-sonarlint now support cfamily languages. At last, I update the render function of "Show Rule Description". It will bring more details of help information.

xeonxu and others added 2 commits September 21, 2023 20:36
…ly. And enables cfamily support as well.

Update version to 0.0.2.
@xeonxu
Copy link
Author

xeonxu commented Sep 21, 2023

Screenshot 2023-09-21 210931
Here is a C file with lsp-sonarlint enabled.

@Horrih
Copy link
Contributor

Horrih commented Jun 8, 2024

Hello @xeonxu
I just realized that I implemented a PR really similar to yours.
Any news from maintainers about your PR ?

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.

2 participants