lsp

:clojure-lsp: Clojure implementation of the Language Server Protocol: https://clojure-lsp.io/
ericdallo 2021-01-26T14:13:22.105Z

@brandon.ringe With this https://github.com/clj-kondo/clj-kondo/pull/1147 fix we now show correctly the range for textDocument/documentSymbol . About the returned symbols, I'm thinking that the first element (always the current ns/file) does not makes much sense, what about showing only the functions/vars?

ericdallo 2021-01-26T14:13:50.105100Z

For example, in emacs we show unnecessarily the ns again:

ericdallo 2021-01-26T14:14:22.105500Z

I think file -> function it'd be enough, instead of file -> my-ns -> function WDYT?

bringe 2021-01-26T17:04:01.106200Z

That sounds good to me

bringe 2021-01-26T17:04:10.106400Z

Can try it out at least and make sure it looks good

ericdallo 2021-01-26T17:04:36.106600Z

Ok, I'll open a PR for you to test it, ok?

bringe 2021-01-26T17:05:31.106800Z

Great!

ericdallo 2021-01-26T17:40:31.107Z

:reviewplease: https://github.com/clojure-lsp/clojure-lsp/pull/281

👍 1
ericdallo 2021-01-26T22:38:28.109200Z

:clojure-lsp: Released https://github.com/clojure-lsp/clojure-lsp/releases/tag/2021.01.26-22.35.27 version of clojure-lspimproving memory usage 🎉, let me know if you find any issues.

4