BetterThanTomorrow / calva

Clojure & ClojureScript Interactive Programming for VS Code
https://marketplace.visualstudio.com/items?itemName=betterthantomorrow.calva
Other
1.67k stars 217 forks source link

clean-ns and other refactor commands not working (windows) #1350

Open kstehn opened 3 years ago

kstehn commented 3 years ago

hey since version 2.0.217 the clean-ns command doesn't work anymore. When executing this nothing happens, no error to see and no changes made.

In Version 2.0.216 this works correctly.

Tested only with windows.

PEZ commented 3 years ago

Very odd. How about in the latest version? 219 as of this writing.

kstehn commented 3 years ago

Same tested this before writing the issue. Just forgot to mention it.

kstehn commented 3 years ago

Just tested this within a wsl instance of vs-code. There it works correctly. Maybe this helps somehow.

PEZ commented 3 years ago

Do the other refactorings work?

kstehn commented 3 years ago

dont use most of them. but no nothing works.

There are no errors in the lsp.log

ericdallo commented 3 years ago

@bpringe how hard is to include the clojure-lsp log inside calva? I think it should help a lot debugging in general I mean this log: https://clojure-lsp.io/troubleshooting/#server-log

bpringe commented 3 years ago

@ericdallo By "include," do you mean include a way to open the file easily from VS Code? We could add a command to open the clojure-lsp log file, which uses the server info command to get the path, right?

ericdallo commented 3 years ago

exactly! Yes, that'd help a lot everyone including us :)