lsp

:clojure-lsp: Clojure implementation of the Language Server Protocol: https://clojure-lsp.io/
ericdallo 2021-04-07T17:20:44.253200Z

:clojure-lsp: Released https://github.com/clojure-lsp/clojure-lsp/releases/tag/2021.04.07-16.34.10: • Fix renaming keywords in cljc files producing duplicate edits. @rvlo • After project startup, publish all project only diagnostics to client. This is a approach done by other LSPs to make work features like https://emacs-lsp.github.io/lsp-mode/page/main-features/#project-errors-on-modeline. Feature flag via lint-project-files-after-startup? with default true. • Add experimental support for aarch64 linux native binary (the CI pipeline did't work yet, need to investigate @brandon.ringe ) • Fix formatting issues with a regression introduced on a previous release. - Fixes https://github.com/clojure-lsp/clojure-lsp/issues/399 and https://github.com/clojure-lsp/clojure-lsp/issues/396 c/c @nicdaoraf Any issues with wrong LSP formatting/indentation, this release should fix it

👍 2
🎉 4
anonimitoraf 2021-04-07T21:18:45.254Z

Thanks!

jcsims 2021-04-07T21:39:59.255600Z

I noticed what looks like a bug here: https://github.com/emacs-lsp/lsp-mode/blob/master/clients/lsp-clojure.el#L217-L218, where concat is used instead of e.g. f-join. This results in a missing path separator for the second item in the list (where lsp-workspace-root returns a path without a trailing slash), and the result is a path something like "/Users/jcsims/code/kondo-test.clj-kondo/config.edn" instead of "/Users/jcsims/code/kondo-test/.clj-kondo/config.edn"

jcsims 2021-04-07T21:40:18.256200Z

I think f-join is the right thing there - ok if I open a PR to that affect?

ericdallo 2021-04-07T22:16:07.256300Z

yes, you are right 🙂

ericdallo 2021-04-07T22:16:19.256500Z

feel free to open an PR on lsp-mode!

👍 1
jcsims 2021-04-07T23:07:54.256800Z

Opened! https://github.com/emacs-lsp/lsp-mode/pull/2768

👍 1