Fork me on GitHub
#clj-kondo
<
2022-11-02
>
borkdude11:11:51

Changelogs for release later today: https://github.com/clj-kondo/clj-kondo/blob/master/CHANGELOG.md#unreleased Please give it a spin on your repo before release, as a sanity check.

🎉 1
clj-kondo 1
clojure-spin 1
Noah Bogart15:11:46

$ ~/Downloads/clj-kondo-2022.10.15-SNAPSHOT-linux-amd64/clj-kondo --parallel --cache false --lint src/ dev/ test/
linting took 12783ms, errors: 0, warnings: 0
lookin good to me (using the artifact from https://github.com/clj-kondo/clj-kondo/commit/7798e4ecbdf65495f2bf25f58df2d2ba0540fc8f)

Sam Ritchie20:11:57

hey all! I am trying to get the linter to totally ignore arguments to a macro, since I haven’t yet written a hook, and thought this might work:

{:lint-as {jsxgraph.clerk-ui/cljs clojure.core/comment}}

Sam Ritchie20:11:23

no dice. any advice on why this wouldn’t work, or if there is a :lint-as solution here? or if I should just write a hook that ignores everything

borkdude21:11:58

@sritchie09 clj-kondo doesn't just ignore anything in comment forms since it's quite useful to have linting in your comment forms. You can tweak what linting you get in comment forms using :config-in-comment - similarly you can tweak what config you get in calls to your macro with :config-in-call See docs https://github.com/clj-kondo/clj-kondo/blob/master/doc/config.md#override-config-in-comment-forms

❤️ 1
Sam Ritchie21:11:20

actually @borkdude maybe there is a way to do this without a hook. This macro should emit nothing for clj , and everything passed to it for cljs; is there some default way to specify this?

borkdude21:11:49

I'll get back to you tomorrow đź’¤

🙏 1
borkdude14:11:36

I'm going to look at this in 20 minutes

Sam Ritchie15:11:30

@borkdude if you haven’t looked, I am now past this;

Sam Ritchie15:11:03

I figured out how to skip compilation of notebooks, so now that macro only runs on the JS side and can’t do any JS linting anyway, so I will go with the ignore style that you showed me. thank you!