This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-12-19
Channels
- # adventofcode (150)
- # aleph (15)
- # announcements (1)
- # beginners (52)
- # cider (3)
- # clj-kondo (2)
- # cljsrn (2)
- # clojure (49)
- # clojure-chicago (1)
- # clojure-europe (13)
- # clojure-france (20)
- # clojure-spec (3)
- # clojure-uk (1)
- # clojurescript (16)
- # community-development (3)
- # core-matrix (1)
- # cursive (9)
- # defnpodcast (1)
- # deps-new (43)
- # fulcro (68)
- # graphql (1)
- # malli (7)
- # off-topic (25)
- # portal (4)
- # re-frame (15)
- # reagent (7)
- # releases (1)
- # remote-jobs (1)
- # rewrite-clj (8)
- # shadow-cljs (4)
- # vim (22)
Heya @vemv, my https://github.com/lread/rewrite-cljc-playground/tree/lread-ns-kw-map is probably ready for a whirl.
My focus was on reworking sexpr, so I’m not sure that I did https://github.com/lread/rewrite-cljc-playground/issues/7#issuecomment-667743070 (but note that I did rename :namespaced? to :auto-resolved? on the keyword node record).
I did not update my design notes yet, but did rework https://github.com/lread/rewrite-cljc-playground/blob/lread-ns-kw-map/doc/01-introduction.adoc which walk through some examples.
I’m very happy to hear any feedback you might have.
🙌 I might give it a spin tomorrow. Just yesterday I was making sure I could spin a repl + run tests in the project :)
Slowly diving into it :)
I'm studing a bit the tests. Regarding the use of clojure.test/are, when there's a do or let in are's "expr" argument, it's nice to place a true at tail position.
That way, if the test fails, it only does once, instead of twice (inner is clauses + implicit surrounding is clause)
I also spotted an are with a single is, which is redundant (you can just place = instead of is =)
