This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-10-18
Channels
- # announcements (18)
- # babashka (34)
- # beginners (47)
- # biff (1)
- # calva (99)
- # cider (1)
- # clojure (99)
- # clojure-austin (13)
- # clojure-australia (1)
- # clojure-dev (53)
- # clojure-europe (30)
- # clojure-nl (1)
- # clojure-norway (7)
- # clojure-spec (7)
- # clojure-uk (1)
- # clojurescript (50)
- # cursive (11)
- # datahike (20)
- # datomic (10)
- # fulcro (7)
- # graalvm (8)
- # gratitude (1)
- # introduce-yourself (7)
- # jackdaw (1)
- # joyride (1)
- # lambdaisland (7)
- # lsp (2)
- # malli (7)
- # nbb (1)
- # off-topic (1)
- # portal (24)
- # re-frame (8)
- # reagent (13)
- # reitit (7)
- # releases (2)
- # remote-jobs (8)
- # rewrite-clj (3)
- # sci (1)
- # scittle (21)
- # shadow-cljs (2)
- # testing (3)
- # tools-deps (22)
- # web-security (19)
@ikitommi I'm writing some tests for
and friends and noticed that the arguments are a little confusing.
start!
takes :ns
which indicates the set of namespaces to be collected, but then instrument!
doesn't use that set - instead you have to separately pass :filters
with [(mi/-filter-ns...)
do you think it makes sense to use the set of namespaces passed in to start!
if they are present and pass those to filters so instrument!
will use the same set? Otherwise you have to pass both :ns
and :filters
to only start collection and instrumentation on the desired set of namespaces
:thinking_face: just looked at the code, dev/start!
doesn't seem to use any :ns
option either. Where did you pick that up?
oh wow, that's my mistake!
I see what happened - it's collect!
that uses it,
https://github.com/metosin/malli/blob/546eb663484e66ef1271b99189b17dbeca215ec8/src/malli/dev.clj#L23
which does support changing what is collected:
https://github.com/metosin/malli/blob/546eb663484e66ef1271b99189b17dbeca215ec8/src/malli/instrument.clj#L46
I must have saw that and inferred that you wouldn't always want to collect everything when using dev/start!
do you have any thoughts on if the :ns
option should be supported for start!
? I can change it back for the cljs version if you want