This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-03-19
Channels
- # announcements (6)
- # aws (10)
- # beginners (73)
- # bristol-clojurians (2)
- # calva (9)
- # cider (25)
- # clj-kondo (7)
- # clojure (160)
- # clojure-dev (2)
- # clojure-europe (63)
- # clojure-italy (7)
- # clojure-nl (10)
- # clojure-uk (76)
- # clojuredesign-podcast (6)
- # clojurescript (63)
- # cursive (6)
- # data-science (3)
- # datomic (26)
- # duct (59)
- # emacs (1)
- # fulcro (12)
- # graalvm (17)
- # hoplon (23)
- # jobs-discuss (2)
- # kaocha (6)
- # meander (7)
- # off-topic (3)
- # pathom (2)
- # rdf (68)
- # re-frame (12)
- # reagent (20)
- # reitit (5)
- # ring (3)
- # ring-swagger (1)
- # shadow-cljs (14)
- # spacemacs (10)
- # sql (3)
- # tools-deps (30)
- # yada (9)
I noticed after a discussion elsewhere today that kondo will lint if
but does not seem to lint if-not
. I think the linting rules would be the same, so this should be an easy change. I can submit a pull request this weekend if we want this
should have checked there. I can try my hand at it. Is there a name you like for #{if if-let if-not ...}
? branching-conditionals
?
there is a giant case expression in analyzer.clj
. it has one case like this:
if (analyze-if ctx expr)
if that's changed to (if if-not)
then I think it works for if-not
as wellthe other one is:
(if-let if-some when-let when-some when-first)
(analyze-conditional-let ctx resolved-as-clojure-var-name expr)
but analyze-conditional-let
doesn't check for missing else branches. it can be easily added.Nice, that is where I was looking. I'll be able to spend a little time on it on Saturday. Thanks for the awesome tool