This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-06-17
Channels
- # announcements (2)
- # aws (4)
- # babashka (20)
- # beginners (256)
- # calva (15)
- # chlorine-clover (1)
- # cider (12)
- # clj-kondo (25)
- # cljsrn (16)
- # clojure (115)
- # clojure-europe (7)
- # clojure-finland (5)
- # clojure-italy (10)
- # clojure-nl (35)
- # clojure-spec (13)
- # clojure-uk (83)
- # clojurescript (42)
- # code-reviews (81)
- # conjure (8)
- # cursive (6)
- # data-science (1)
- # datomic (3)
- # duct (18)
- # emacs (2)
- # figwheel-main (40)
- # fulcro (21)
- # helix (1)
- # jackdaw (7)
- # java (2)
- # lein-figwheel (5)
- # luminus (1)
- # observability (12)
- # off-topic (32)
- # parinfer (5)
- # pedestal (19)
- # re-frame (5)
- # reagent (8)
- # reitit (17)
- # rewrite-clj (47)
- # rum (19)
- # shadow-cljs (48)
- # spacemacs (4)
- # sql (40)
- # tools-deps (22)
- # vrac (1)
- # xtdb (25)
I'm soon resuming work on the macroexpand feature and would still very much like feedback on this. https://clojureverse.org/t/feedback-wanted-on-new-clj-kondo-macroexpansion-feature/6043
In a different issue someone suggested it would be useful to have a hook in the function call so you can lint arguments to some function yourself. This would be a generalization of the macroexpand feature as it is proposed now. E.g. with re-frame, a dispatch function should get a qualified keyword. This is something people can't do with the analysis output today (although that could also be extended with the sexpr).
one of the problems in the above example is that keywords and numbers don't carry metadata
also I think rather than throwing people can return :findings
with :level
, :row
, :col
etc.
The row and col come from the metadata of the sexpr elements, but in case of numbers of keywords these aren't present
Of course the function could get an additional :node
thing which is the raw rewrite-clj node
this is probably already a power-feature, so the raw rewrite-clj node is probably good to start with
I'm not sure you can close 468 from this, as custom linters might be broader, like less than 10 functions per namespace.