This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-02-26
Channels
- # announcements (18)
- # aws (17)
- # babashka (19)
- # beginners (141)
- # calva (73)
- # cider (4)
- # clj-kondo (13)
- # cljs-dev (2)
- # clojure (97)
- # clojure-europe (6)
- # clojure-italy (5)
- # clojure-nl (1)
- # clojure-spec (25)
- # clojure-sweden (2)
- # clojure-uk (25)
- # clojured (3)
- # clojurescript (63)
- # core-typed (6)
- # cursive (23)
- # data-science (4)
- # datomic (74)
- # fulcro (19)
- # graalvm (18)
- # graphql (3)
- # hoplon (63)
- # jackdaw (1)
- # juxt (23)
- # london-clojurians (3)
- # meander (7)
- # off-topic (23)
- # om (1)
- # pathom (13)
- # pedestal (2)
- # perun (2)
- # re-frame (38)
- # reagent (3)
- # reitit (24)
- # shadow-cljs (91)
- # spacemacs (14)
- # sql (4)
- # tools-deps (8)
- # vim (3)
Hey, I am a huge fan of kondo and have set it up in all my clojure projects 🙂
I was wondering if there plans to add fixers to automatically fix some of the linting errors ?
I’ve just set it up on an existing project and the output is errors: 151, warnings: 421
😿 so it’d be really cool if it can do some of the work for me
Thanks! Carve is an example of a project that uses clj-kondo AND fixes stuff automatically in your code. I suggest you take a look at that. Clj-kondo will be linting only.
I’ve been meaning to try Carve since you first announced it but honestly I forgot… this chat reminded me that it exists. I just tried it out on my project and wow that’s great. Thank you!
Given this in config.edn
: :lint-as {com.rpl.proxy-plus/proxy+ clojure.core/defrecord}
, using proxy+
does not show interface/class symbols as used. So for example:
(proxy+ []
Combine$CombineFn
(createAccumulator
[_]
...)
will still give a warning that Combine$CombineFn
is unused. Not a big deal nor very surprising, since it would seem to hard to cover this in general, just thought I'd mention it in case there was some other way I should configure or something.You're right, proxy+
doesn't have a name, probably shouldn't have expected it to work in the first place. proxy+
also doesn't have the same syntax as proxy
, though I'll give it a try anyway.
Using reify
instead of defrecord
did the trick. Nice!