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

Feature request: support coq_nvim (or: be autocompletion-engine-agnostic) #34

Closed
neumannjan opened this issue May 13, 2023 · 2 comments
Closed
Labels
enhancement New feature or request

Comments

@neumannjan
Copy link

Hi,

I love the plugin, but recently I switched away from nvim-cmp to coq_nvim due to performance reasons and unfortunately it seems that otter lacks support for coq.

Could support for coq please be added?

Alternatively, why not register a new LSP directly in neovim (in lua) which will simply redirect to other relevant LSPs, instead of having this implementation as a separate autocompletion plugin? I haven't looked into neovim APIs in great detail, but it seems that something along those lines should be possible.

Thanks again for the plugin and your time.

@jmbuhr
Copy link
Owner

jmbuhr commented May 13, 2023

I agree that a language server that performs essentially the functionality of otter.nvim (keep pure language buffers in sync, redirect requests) would be a very clean solution, though it is quite likely to find a bunch of edgcases in lsp and neovim api implementations that may make it less clean (hence I tend to avoid the word "simply" when programming ;) ). However, I don't currently have the time to write a language server.

As for coq, it looks like a very promising approach and seems to also be extensible similar to cmp. I would be very happy about a PR to otter.nvim if you translate the cmp source there to coq.

@jmbuhr jmbuhr added enhancement New feature or request help wanted Extra attention is needed labels May 13, 2023
@jmbuhr jmbuhr removed the help wanted Extra attention is needed label May 29, 2023
@jmbuhr
Copy link
Owner

jmbuhr commented Jul 14, 2023

I will revisit this once neovim/neovim#24338 or something along those lines is merged. This should make it easier to turn otter.nvim essentially into a language server with the necessary translations included.

@jmbuhr jmbuhr closed this as completed Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants