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

Show formatters and linters instead of null_ls #25

Closed
tnthi115 opened this issue Dec 16, 2023 · 8 comments · Fixed by #26
Closed

Show formatters and linters instead of null_ls #25

tnthi115 opened this issue Dec 16, 2023 · 8 comments · Fixed by #26

Comments

@tnthi115
Copy link

Instead of showing null_ls, it would be nice to see which formatters and linters are attached to the buffer like in Lunarvim or Astronvim.

image

@vladimir-popov
Copy link
Collaborator

I suppose, it would be better to create a separate component specially for your case. I don't use null_ls, but I'll try to look into it when I will have time

@vladimir-popov
Copy link
Collaborator

@tnthi115 according to this issue jose-elias-alvarez/null-ls.nvim#1621 null-ls is not supported. Do you know an actual fork?

@vladimir-popov
Copy link
Collaborator

vladimir-popov commented Dec 21, 2023

Ok, I've found the answer. The LunarVim already moved to none-ls.nvim: LunarVim/LunarVim#4392.

@tnthi115
Copy link
Author

This snippet is probably helpful: https://gist.github.com/Lamarcke/36e086dd3bb2cebc593d505e2f838e07

@vladimir-popov
Copy link
Collaborator

@tnthi115 check please this PR, is it ok for you, or maybe you have any ideas? #26

@tnthi115
Copy link
Author

@vladimir-popov Great work, thank you! Have you considered extending the ex.lsp.all list instead of having a separate component and list for null_ls alone?

@vladimir-popov
Copy link
Collaborator

@tnthi115 I'm still thinking that separate components are better and more flexible. For instance, I find it preferable to utilize the ex.lsp.all component in the only_icons mode. The specific names of LSP servers are less critical to me. Conversely, when dealing with null-ls, it is reasonable to see the full names of actual sources. So, I personally hide an icon for null-ls in the ex.lsp.all component by setting empty string to it, and it looks like this:
Screenshot 2023-12-27 at 12 51 00

Maybe you have use-case in which separate components are not appropriate to you?

@tnthi115
Copy link
Author

@vladimir-popov I think your example works well. Thank you for your hard work!

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 a pull request may close this issue.

2 participants