Skip to content

Conversation

@kiennq
Copy link
Contributor

@kiennq kiennq commented Feb 16, 2025

Support official Github copilot LSP and allow to install the latest LSP version via copilot-version = nil.

@zerolfx zerolfx merged commit 6fc7307 into copilot-emacs:main Feb 17, 2025
10 checks passed
@Kaylebor
Copy link

Kaylebor commented Feb 17, 2025

I've been trying to have copilot reference a local install of the LSP, like this on my config.el (Doom Emacs):

(after! copilot
  (setq copilot-server-executable (expand-file-name "~/.bun/bin/copilot-language-server")))

I have also tried:

(use-package! copilot
  :after copilot
  :config (setq copilot-server-executable (expand-file-name "~/.bun/bin/copilot-language-server")))

The variable is correctly set when checking its value, but copilot.el complains that there's no server installed.
Is the only supported method to use the package-installed LSP, and never a globally-available one? With or without this variable, it fails to find it.

@kiennq
Copy link
Contributor Author

kiennq commented Feb 17, 2025

I've been trying to have copilot reference a local install of the LSP, like this on my config.el (Doom Emacs):

(after! copilot
  (setq copilot-server-executable (expand-file-name "~/.bun/bin/copilot-language-server")))

I have also tried:

(use-package! copilot
  :after copilot
  :config (setq copilot-server-executable (expand-file-name "~/.bun/bin/copilot-language-server")))

The variable is correctly set when checking its value, but copilot.el complains that there's no server installed. Is the only supported method to use the package-installed LSP, and never a globally-available one? With or without this variable, it fails to find it.

Should be fixed in #364

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.

5 participants