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

doc: add WIP nvim example config #17

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Robzz
Copy link

@Robzz Robzz commented Jun 10, 2024

This PR adds an example configuration for neovim.

This to do before this is ready for merging:

  • Contribute a default configuration to the nvim-lspconfig repository. On top of providing a sane default config, this allows automated proper management of the language server lifecycle, because to be clear the provided sample code kinda smells: never detaches closed buffers, never shuts the LSP down, you get the idea, it's naive.
  • Wait until things settle down on the nvim-cmp side. Support for multiline ghost-text was literally merged 2 days ago, and there are additional PRs addressing limitations that are actively being worked on, such as the suggestions window currently being drawn on top of the ghost text (PR 1955). In its current state, it's a bit rough around the edges, but it should be good enough to play with for the curious.
  • Register the textDocument/generation function. I still need to look into how that works, but the nvim-lspconfig docs point towards setting up an LspAttach autofunction handler with vim.api.nvim_create_user_command()

Closes #2

@Robzz
Copy link
Author

Robzz commented Jun 10, 2024

I added an lspconfig based example using my fork (happy to send the PR their way once we're happy with it).

@SilasMarvin
Copy link
Owner

Just added a comment in this thread: #2 (comment)

Excited to get this into nvim-lspconfig!

@Robzz
Copy link
Author

Robzz commented Jun 18, 2024

PR opened yesterday, took a bit longer than I would have liked but time is what it is. Waiting on me writing a bit of docs.

@SilasMarvin
Copy link
Owner

PR opened yesterday, took a bit longer than I would have liked but time is what it is. Waiting on me writing a bit of docs.

This looks awesome thank you! Let me know if I can assist at all

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.

Neovim Plugin and nvim-lspconfig integration
2 participants