-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Support for neovim/nvim-lspconfig #21
Comments
It looks like related to #10? |
Yes, it is! I was happy to see it's already supported by Mason, it was a pleasant surprise. In theory Mason support is not required to use lspconfig, but in practice everyone depends on Mason, so without it we'll need some hacks and non-standard setup. |
Mason also does not work particularly well on Termux itself, |
Not sure I understand what you mean. If there are issues with using Mason itself then all you need to use lspconfig (as shown above) without Mason is to make sure your nvim config won't call some mason plugin (e.g mason itself or 'williamboman/mason-lspconfig.nvim' or 'WhoIsSethDaniel/mason-tool-installer.nvim') to install termux-language-server (e.g. with 'termux_ls' item inside 'ensure_installed' list). Of course, in this case you'll have to install termux-language-server manually. |
|
@powerman What file do I add this segment to? I'm trying to get this to work for me. I'm an AUR maintainer, looking to get initiated as an official Arch packager. |
@txtsd You can see my complete config with this snippet here: https://github.com/search?q=repo%3Apowerman%2Fconfig.nvim+termux&type=code |
@powerman I used the relevant bits and incorporated it in my lazyvim setup, but it shows my entire PKGBUILD as an error. Is this something you're familiar with? |
Nope. If Neovim LSP works but show unexpected issues in your sources then open a separate issue about this. |
Thanks for this example config! I'm just wondering if it wouldn't be a better approach to check with nvim-lspconfig's contribution guidelines and then just make a pull request directly over there. This server seems pretty stable by now, and even Mason supports it. On another note, does this server even need a |
While recommended configuration for Neovim at https://termux-language-server.readthedocs.io/en/latest/resources/configure.html#neovim works it's inconvenient to use together with lspconfig. I propose to add lspconfig support for termux-language-server.
I've already implemented and tested most of it (I've no idea is it exist and if yes then how to detect a "root dir" for Android Termux and ArchLinux projects, so I leave a TODO comment there). Probably needs some polishing and then you can make a couple PRs:
Here is setup I've used for testing.
~/.config/nvim/lua/lspconfig/server_configurations/termux_ls.lua
:That's it, the rest of setup is usual for lspconfig.
The text was updated successfully, but these errors were encountered: