lsp-mode icon indicating copy to clipboard operation
lsp-mode copied to clipboard

Astro language server not showing in lsp-install-server

Open bustinbung opened this issue 2 years ago • 2 comments

Thank you for the bug report

  • [X] I am using the latest version of lsp-mode related packages.
  • [X] I checked FAQ and Troubleshooting sections
  • [X] You may also try reproduce the issue using clean environment using the following command: M-x lsp-start-plain

Bug description

When trying to install the Astro language server through lsp-install-server, astro-ls does not show up. According to @hlissner on the Doom Emacs discord, it's because there is no configuration in lsp-astro.el. (source)

Steps to reproduce

  • Install lsp-mode
  • Run lsp-install-server
  • Search for astro-ls
  • No entry

Expected behavior

  • astro-ls should show in the list of servers, as stated in the documentation page

Which Language Server did you use?

astro-ls

OS

MacOS

Error callstack

No response

Anything else?

I also got this error, but I'm not sure it's related to this issue. ../.emacs.d/.local/straight/build-29.1/lsp-mode/lsp-mode.el: Warning: Non-symbol arguments to cl-defgeneric: (process process) [5 times]

bustinbung avatar Sep 11 '23 19:09 bustinbung

To spare you the trouble of hunting on our Discord server, these were my observations: the client defined in clients/lsp-astro.el lacks a :download-server-fn property and/or a lsp-dependency declaration (which I assume is the bare minimum needed for a client to show up in M-x lsp-install-server). Yet, lsp-mode's docs for Astro suggest astro-ls is installable via M-x lsp-install-server.

hlissner avatar Sep 11 '23 19:09 hlissner

Yeah, @hlissner is correct, tagging latest people that worked on that support: @tian-yi @lucafanselau @pawelkobojek. @bustinbung feel free to open PRs if you know how lsp-mode can install astro

ericdallo avatar Sep 11 '23 21:09 ericdallo

Bumping this issue. Will look into PRing an installer, but at the very least, misleading docs should be addressed

jacksonSingleton avatar Feb 24 '24 16:02 jacksonSingleton

Anyone know how to install? I'm happy to update docs. Do I just... pop this PR code into my config or something? https://github.com/emacs-lsp/lsp-mode/pull/3816

komali2 avatar Apr 03 '24 12:04 komali2

Something like this? https://github.com/emacs-lsp/lsp-mode/pull/4414

komali2 avatar Apr 03 '24 13:04 komali2

It seems like you got it. I am closing this feature request now.

jcs090218 avatar Apr 03 '24 18:04 jcs090218