Fork me on GitHub
#calva
<
2021-11-08
>
pez10:11:00

I’m having that semantic token highlighting error now. (After a “git stash save -> get rebase master -> git stash pop”). It’s visible in the screenshots in two ways. The db is a local variable, but highlighted as a global definition. :config has funny dual coloring. The second screenshot is from the same file, in another form where the db variable looks OK. The semantic markup looks OK. Still not sure if the issue is with #lsp , VS Code, or Calva’s lsp client setup. CC: @ericdallo

ericdallo12:11:52

it's totally a lsp issue, but a known issue that I intend to fix it soon

ericdallo12:11:04

the issue happens when changing files outside the editor, like git branch changes

ericdallo12:11:09

we currently ignore the didChangeWatchedFiles notifications of type change because last time I added support for that it reduced the performance, then I removed and left to fix it later

ericdallo12:11:51

feel free to create an issue on clojure-lsp side about analysis outdated when changing files outside the editor

pez12:11:51

Ah, cool. I might add that issue. 😃

Stefan10:11:58

I have also seen something that is maybe the same bug when using live share. Sometimes (not sure when) highlighting is broken, and re-joining the live share session then fixes it. I think closing all files and then reopening them fixes it as well, but I’m not 100% sure about that one.

pez11:11:10

LiveShare might be a similar situation, that the files are changed from “underneath”.

pez18:11:24

I’m starting to think that parinfer is coming to Calva. https://twitter.com/pappapez/status/1457646855802740737?s=21

pez18:11:38

Please consider installing the vsix from that PR (in the start of the Twitter thread). I'd especially like to know whether it works as before with the defaults (parinfer off).

pez23:11:51

Dear Calva friends. I need help testing the Parinfer VSIX. I’d like to know that it won’t cause too much troubles for people not enabling the experimental features, so even if you are not interested in Parinfer, I’d appreciate your help. Here’s the VSIX: https://15890-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.223-pez-253-parinfer-experiments-ad2037dd.vsix And here’s a demo. And here are docs: https://github.com/BetterThanTomorrow/calva/blob/pez/253-parinfer-experiments/docs/site/parinfer.md

👍 1
bringe04:11:10

I’ve installed it and will use it while working with the experimental features disabled.

🙏 1
gratitude 1
Jakub Holý (HolyJak)11:11:29

update the link in the orig. message also? 🙏

1
🙏 1
emccue18:11:33

Parinfer is a required feature for me to be happy and this times up well with us testing calva for the engineering team writ large

pez21:11:43

RC3: https://15890-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.223-pez-253-parinfer-experiments-ad2037dd.vsix Fixing an issue with misplaced cursor after some edits. I think I am doing something wrong since I need this fix, I’ve reached out in #parinfer about it.

bringe05:11:40

No issues to report today while working without the experimental features on. I just installed RC3. 🚀

❤️ 1
metal 1