calva Add Missing Require refactoring includes metadata

Thank you @bpringe, @ericdallo for coming up the fix for clojure-lsp on Windows!

I just tried it, noticed a minor issue. Add Missing Require included metadata in require.

Dec 31 '20 03:12 duke13137

I have also seen this happen in dev, when I added the refactorings. But then it stopped behaving like that, and I thought I had found the right settings to prevent it...

Dec 31 '20 09:12 PEZ

Never saw this behaviour before 🤔

Dec 31 '20 13:12 ericdallo

https://github.com/clojure-lsp/clojure-lsp/issues/120 Here's a related issue but it looks like it was believed to be fixed. However, it must not be.

Dec 31 '20 19:12 bpringe

I could not reproduce these issues, do you have a sample where I can reproduce it using lsp-mode?

Dec 31 '20 19:12 ericdallo

Might just be on Windows. I'll test when I can get to this, if no one else does before me.

Dec 31 '20 20:12 bpringe

I can't reproduce with calva 2.0.143 + clojure-lsp 2020.12.31-17.22.22 on windows anymore. It's a low priority.

Dec 31 '20 20:12 duke13137

It is not just on Windows. I've seen it on Mac. Now idea how to reproduce it though. But it is very probably upstream (clojure-lsp or beyond).

Jan 03 '21 21:01 PEZ

It can be an issue on clojure-lsp, but I have never seen that, and it's quite hard fix it without a sample to reproduce it.

Jan 03 '21 21:01 ericdallo

Indeed. I will update if I encounter this again. But as things stand now, none of us can't really do anything about it.

Jan 03 '21 21:01 PEZ

Has anyone seen this in a while? If not, I think we can close this.

Jan 22 '23 22:01 bpringe