Paulus Esterhazy
Paulus Esterhazy
Made a stab at this https://github.com/lambdaisland/kaocha/pull/321
For future reference, some pointers to CIDER's source code that could be useful: https://github.com/clojure-emacs/cider-nrepl/blob/master/src/cider/nrepl/middleware/info.clj#L87 https://github.com/clojure-emacs/cider-nrepl/blob/master/src/cider/nrepl/middleware/complete.clj
I'm still running into (what I assume is) this error. In case it's not clear how to repro: Steps to repro 1. kaocha --watch (one test file) 2. see tests...
Note that if I restart the process, I do see the error - the error is swallowed only after reload
## before 76 lines ``` ERROR in unit (pico/app/control_test.clj:18) Failed loading tests: Exception: clojure.lang.Compiler$CompilerException: Syntax error compiling at (pico/app/control_test.clj:18:3). #:clojure.error{:phase :compile-syntax-check, :line 18, :column 3, :source "pico/app/control_test.clj"} at clojure.lang.Compiler.analyze (Compiler.java:6825)...
> Would there be a way to opt-out of this behavior? Totally agree, for some cases - e.g. working on kaocha itself - it might be preferable to see the...
With the optional support, we could do "staged rollout": it could start out as optional and then be promoted to default when people like it
The above was tested using non-graal binaries. With graal binaries, the behavior is even more striking - https://github.com/clojure-lsp/clojure-lsp/releases/tag/2022.02.01-16.53.14 or newer releases: Change Coll doesn't work even once - https://github.com/clojure-lsp/clojure-lsp/releases/tag/2022.01.22-01.31.09: Works...
OK so on clojure-lsp 2022.01.22-01.31.09 - code action "covert to set" (list, map, etc) works reliably - lsp-clojure-cycle-coll doesn't do anything mostly (I did see it work once though but...
Hi @xaris as should be obvious by now, I haven't had a chance to work on this. It's obviously quite a lot of work. As you say it would be...