the latest nightly lsp contains a fix for the NPE

Nice!!

(which is in clj-kondo which was just bumped)

but still it might be better to not use def-catch-all for something it wasn't built for

{:config-in-call {foo/bar {:linters {:ignore true}}}}

is the better option, you can also provide :ignore [:linter1 :linter2]

OK, will try that for defphraser once I've had a coffee... 🙂

...confirmed that the NPE is gone 🙂 Updating that config now...

Where should that :config-in-call go? I tried it in the main .clj-kondo/config.edn for the project and got a bunch of unresolved symbol errors from the defphraser calls that I don't get for def-catch-all.

yes, in .clj-kondo/config.edn, what does it look like now?

Here's what I had (which "works"):

:lint-as
 {phrase.alpha/defphraser clj-kondo.lint-as/def-catch-all}

:config-in-call
 {phrase.alpha/defphraser {:linters {:ignore true}}}

(defphraser #(contains? % k)
  [_ _ k]
  (str (name k) " is required."))

I'll try to repro...

ah, it should be:

:config-in-call {defphraser/defphraser {:ignore true}}

so without the :linters indirection

Yup, that fixed it!

VS Code's Explorer now shows no (lint) errors in our entire code base 🙂

We have a couple of def-catch-all in the config... one is definitely a def-like thing... I'll have to look at the other one...

If I want a specific (comment ..) form to not be linted...?

just one?

#_:clj-kondo/ignore
(comment x y z)

Yup, just one. Thanks.