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

Can this be a part of lsp-mode? #1

Open
kiennq opened this issue Nov 6, 2024 · 1 comment
Open

Can this be a part of lsp-mode? #1

kiennq opened this issue Nov 6, 2024 · 1 comment

Comments

@kiennq
Copy link

kiennq commented Nov 6, 2024

Hi @kassick,
I've come across your project and lsp-mode is actually missing the inlineCompletions support now. Would it be possible to contribute this to lsp-mode so that everything will be in a single package, better for user experience?

@kiennq kiennq changed the title Can this be a part of lsp-mode Can this be a part of lsp-mode? Nov 6, 2024
@kassick
Copy link
Owner

kassick commented Nov 6, 2024

Hi!

If the maintainers think that it makes sense to upstream, sure!

(Though, it is mostly UI, other than the kludge to include the inlineCompletions command and its interfaces))

There is some glitch in the response handling that needs fixing -- the lsp-inline-completion function asks for inline completions and normalizes the response. At first, I had both gitlab-lsp and copilot-lsp active and it worked as expected. After I stopped using gitlab-lsp, sometimes the response cached in lsp--inline-completions was not of the expected type . At the time, I did not have enough time to debug it and switched to using upstream copilot.el .

Anyway, this glitch needs to be fixed when upstreaming, but otherwise I'm OK with upstreaming it.

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

No branches or pull requests

2 participants