This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2021-03-11
Channels
- # aleph (50)
- # announcements (20)
- # babashka (68)
- # beginners (70)
- # calva (25)
- # cider (1)
- # clj-kondo (10)
- # cljs-dev (3)
- # clojars (1)
- # clojure (113)
- # clojure-australia (7)
- # clojure-berlin (4)
- # clojure-europe (47)
- # clojure-italy (14)
- # clojure-nl (2)
- # clojure-norway (1)
- # clojure-serbia (5)
- # clojure-spec (11)
- # clojure-uk (8)
- # clojurescript (16)
- # community-development (2)
- # conjure (2)
- # cursive (15)
- # data-oriented-programming (35)
- # datahike (12)
- # datascript (5)
- # datomic (13)
- # duct (7)
- # fulcro (21)
- # graalvm (94)
- # graphql (1)
- # helix (4)
- # honeysql (19)
- # jackdaw (8)
- # jobs (2)
- # jobs-rus (1)
- # leiningen (1)
- # malli (32)
- # missionary (1)
- # mount (1)
- # off-topic (40)
- # perun (2)
- # portal (7)
- # reitit (10)
- # rewrite-clj (26)
- # shadow-cljs (90)
- # spacemacs (29)
- # sql (17)
- # tools-deps (49)
- # wasm (1)
- # xtdb (26)
Awesome, thanks @ericdallo !
@nate Currently a pod isn't possible since the rewrite-clj AST isn't serializable. I'm having thoughts about a separate rewrite-clj
scripting CLI.
@lee Btw you may want to make these tags proper release on Github? https://github.com/clj-commons/rewrite-clj/releases
Why zip/value
is deprecated?
Checking the docstring doesn't point to any replacement
Yeah, was just verifying zip/value
, https://github.com/clj-commons/rewrite-clj/blob/6b359d62ec5bad7d4afe848cd15d116b542c4212/src/rewrite_clj/zip/base.clj#L61. It is really just a call to node.value
https://github.com/clj-commons/rewrite-clj/blob/6b359d62ec5bad7d4afe848cd15d116b542c4212/src/rewrite_clj/node.clj#L115.
Thanks! probably https://github.com/clj-commons/rewrite-clj/blob/6b359d62ec5bad7d4afe848cd15d116b542c4212/src/rewrite_clj/zip/base.clj#L83`zip/string` ?
Or zip/sexpr
, whichever makes more sense in your specific use case. The nice thing about zip/string
, if it works for your case, is that it always works.
yeah, we use z/sexpr
in a lot of places but there are places where the string makes sense
thanks for the help and your work! I'll keep testing the release for a few days, then I'll check if it fixes some clojure-lsp issues with cljs 🤞