Eric Dallo comments

Results 708 comments of


                                            Eric Dallo

Clojure completion of aliased dependencies only works in the REPL

AFAICS this is not a client issue but a clojure-lsp one, but I, as the maintainer, cannot repro it and it's a really basic feature, I suggest closing this one...

compiler command line define interpreted to contain ' ' around '/'

@ville-h This is not a depedency of lsp-mode, it's part of the language server you are using, when using LSP there are always two sides, the client side, the editor...

compiler command line define interpreted to contain ' ' around '/'

I never used clangd or lsp-mode with C so I think I'm really less qualified than you :)

Error on goto-definition for standard library functions

@ajsarama That's odd, it looks like there's something wrong with some file URI as mentioned in the stacktrace, could you try at https://github.com/ericdallo/clojure-sample? Also, would be nice to get the...

Create a separate option to exclude tests from the callers list when listing unused functions

@raszi what about a setting for that linter like `:ignore-tests true`?

Create a separate option to exclude tests from the callers list when listing unused functions

@raszi the idea is to add that under the existing linter, so it would be: `.clj-kondo/config.edn` ```clojure {:linters {:clojure-lsp/unused-public-var {:ignore-tests true}} ``` [ref](https://clojure-lsp.io/settings/#clojure-lspunused-public-var)

Create a separate option to exclude tests from the callers list when listing unused functions

This is done under the `ignore-test-references?` flag, available as a nightly build already

Create a separate option to exclude tests from the callers list when listing unused functions

![image](https://github.com/user-attachments/assets/1f36ed8b-d409-45b9-93d0-01ebf95fccd8)

Create a separate option to exclude tests from the callers list when listing unused functions

@raszi I had to[ revert this feature](https://github.com/clojure-lsp/clojure-lsp/commit/7aefddd1fbb2664b27614f1a6081f75c554213d3) because of a huge decrease in performance, we will work on this fix soon c/c @acamargo

Create a separate option to exclude tests from the callers list when listing unused functions

Yeah, I agree, the problem is that that piece of code is pretty critical to the whole diagnostics performance and it's hard not to affect performance drastically, but we will...