Fork me on GitHub
#calva
<
2023-11-05
>
pez11:11:38

Dear Calva friends: https://github.com/BetterThanTomorrow/calva/releases/tag/v2.0.393 • Fix: https://github.com/BetterThanTomorrow/calva/issues/2336 • Fix: https://github.com/BetterThanTomorrow/calva/issues/2339 Thanks @brjann and @holyjak for reports and reproductions! Thanks @ericdallo for stellar support in getting Calva to use clojure-lsp properly! Thanks @domagala.lukas for squeezing in some Calva support when you were tight on time! ❤️ 🙏 clojure-lsp calva

❤️ 4
🚀 4
DrLjótsson11:11:13

Awesome, thank you so much @U0ETXRFEW !

🙏 1
Jakub Holý (HolyJak)11:11:47

that was quite some speed! thank you!

🙏 1
pez11:11:30

Please let me know wether it is proper fixes. 😃

👍 1
DrLjótsson12:11:54

Works like a charm so far! One thing, which I actually like but may be unintended, is that the option to require as alias has now disappeared for local variables. But if append a slash, suggestions from the aliased namespace appear and if I hit tab, the aliased namespace is required. The reason I think that this is unintended is that autocomplete still suggests aliases for non-local names. But I prefer the "include after slash has been added"-way so I am fine with the change 🙂

🙏 1
Lukas Domagala13:11:08

I agree with you @brjann, it's nice to not accidentally require namespaces when you wanted to use a variable 🙂

pez14:11:05

I have been wondering about why that happens to me so often. Sometimes the require doesn’t even make sense and breaks the namespace. If I have accidentally fixed that, I am extra happy. But probably what really happened was that we had unintended behaviour before, due to the bug.

Jakub Holý (HolyJak)17:11:17

Works like a charm!

metal 1
🙏 1