lsp

:clojure-lsp: Clojure implementation of the Language Server Protocol: https://clojure-lsp.io/
ericdallo 2020-06-21T02:29:41.024900Z

:reviewplease: https://github.com/snoe/clojure-lsp/pull/148

snoe 2020-06-21T17:38:32.032300Z

@ericdallo I'm going to test this out some more tomorrow - want to make sure the lsp4j upgrade doesn't have side effects.

ericdallo 2020-06-21T17:40:17.032500Z

no problem! Thanks for the fast feedback 🙂

ericdallo 2020-06-21T17:41:23.032700Z

@snoe This was the PR that added the most features of 0.9 if you want to take a look: https://github.com/eclipse/lsp4j/pull/397 It helped me to know what to change

snoe 2020-06-21T17:45:37.033Z

thanks

ericdallo 2020-06-24T13:05:43.033400Z

Hey @snoe thanks for the merge! Can we generate a new release?

snoe 2020-06-24T14:19:53.033600Z

hrm, thought i did...

1
ericdallo 2020-06-24T14:21:51.033900Z

The latest is from 4 days 😅

snoe 2020-06-24T14:23:36.034100Z

yeah, I think it's replacing it 😕

😱 1
snoe 2020-06-24T14:30:27.034400Z

ok, fixed; released

ericdallo 2020-06-24T14:31:11.034600Z

Thank you!

dpsutton 2020-06-21T03:07:49.026Z

@ericdallo are you an emacs user? If so i made https://github.com/dpsutton/lsp-clojure-hydra a hydra to use the lsp refactorings. i haven't bothered with getting it into melpa yet but i think it would be a good addition

ericdallo 2020-06-21T14:07:44.026700Z

Yes, doom-emacs user 🙂 Really cool indeed @dpsutton, What do you think about implementing that on https://github.com/emacs-lsp/lsp-mode/blob/master/lsp-clojure.el#L55-L128? Or we can check on lsp-clojure.el if lsp-clojure-hydra.el is available and auto require it to minimize user configuration :) PRs are very welcome on lsp-mode :)

dpsutton 2020-06-21T15:34:36.027200Z

i probably wouldn't want this in lsp-mode for two reasons: 1. i doubt they have or are interested in a dependency on hydra, and 2. it would be much easier to iterate on this at the speed of changes to lsp without having to get a change into lsp, then lsp-mode. Better to put it into a lib that you can approve PRs yourself and release without requiring lsp-mode approval

ericdallo 2020-06-21T15:39:07.027400Z

I understand your concerns but we already have hydra integration for DAP: https://github.com/emacs-lsp/dap-mode/blob/master/dap-hydra.el and a integration for LSP it would be good too 🙂 I can say that the changes on lsp-mode are pretty fast, we are following lsp spec faster than they can provide new features 😛 But if you don't want to implement that on lsp-mode , we can still check if lsp-clojure-hydra package is available then require it automatically 🙂

dpsutton 2020-06-21T15:39:22.027700Z

oh then i'm all for it!

dpsutton 2020-06-21T15:39:38.027900Z

if neither of those sound like blockers i'm absolutely game

dpsutton 2020-06-21T15:39:55.028100Z

and would make it easier for everyone using lsp-mode and clojure-lsp to have it right away

dpsutton 2020-06-21T15:40:09.028300Z

as an aside, are you a regular contributor or core contributor for lsp-mode?

dpsutton 2020-06-21T15:40:19.028500Z

i've seen your name pop up in the gitter a bit now

ericdallo 2020-06-21T15:46:23.028700Z

I'm one of the maintainers 🙂

ericdallo 2020-06-21T15:47:19.028900Z

Questions and suggestions are welcome on gitter channel

dpsutton 2020-06-21T15:49:54.029100Z

awesome! good to know

dpsutton 2020-06-21T03:07:55.026300Z

maybe something you'd be interested in as well

snoe 2020-06-21T05:54:31.026500Z

Looking at it, I think it's less readable on the one line.