fix LSP update logic - Githubissues

NoahTheDuke / coc-clojure

coc.nvim plugin for clojure-lsp

Mozilla Public License 2.0

32 stars 3 forks source link

Closed laur89 closed 1 year ago

laur89 commented 1 year ago

LSP update logic had couple of problems: a) it prompted user for version update before verifying whether it's even needed; b) if update failed or user opted out, then module returned undefined instead of the existing binary path

Fixes NoahTheDuke/coc-clojure#11

Note this change has not been tested!

NoahTheDuke commented 1 year ago

I don't have a ton of time for OS work, but this looks good to me, so let's merge and find out!

laur89 commented 1 year ago

After testing looks good to me. Maybe cut a release?