lite-xl / lite-xl-lsp-servers

Public repo of easily usable lsp servers, that interface with the lite-xl plugin system.
MIT License
8 stars 10 forks source link

Add zig lsp support #5

Closed ArtemisX64 closed 6 months ago

ArtemisX64 commented 6 months ago

Self explanatory

adamharrison commented 6 months ago

Just a heads up; in future, also, please submit a PR to add the stub to the plugins repository here: https://github.com/lite-xl/lite-xl-plugins. I've added it for you this time around tho, so no worries.

adamharrison commented 6 months ago

Also, I uh, just realized that you submitted it with the manifest pointing to lsp_zls.lua, instead of lsp_zig.lua. I fixed that for you. But, uh now we have another problem. When I try to install on linux, I get:

checksum doesn't match for https://objects.githubusercontent.com/github-production-release-asset-2e65be/258636766/52c75dfb-57cf-4285-adf7-86bf0c652e9a?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20231229%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20231229T164912Z&X-Amz-Expires=300&X-Amz-Signature=60307cfd00260eb397056078c2b0072c0dd9233b6eaa0d031c83874dd7f6fe64&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=258636766&response-content-disposition=attachment%3B%20filename%3Dzls-x86_64-linux.tar.gz&response-content-type=application%2Foctet-stream; use --force to override

This generally means your checksum is incorrect. Are you sure you included the right one in the manifest?

adamharrison commented 6 months ago

Ah, I think I see. The issue is you're using md5sums, instead of sha256sums. I'll go correct that for you.

ArtemisX64 commented 6 months ago

:sweat_smile: Thanks for the corrections. This was my first time using lpm.