This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-06-25
Channels
- # announcements (5)
- # babashka (23)
- # beginners (70)
- # cider (24)
- # clj-kondo (14)
- # cljsrn (2)
- # clojars (6)
- # clojure (195)
- # clojure-australia (1)
- # clojure-dev (2)
- # clojure-europe (27)
- # clojure-france (1)
- # clojure-nl (4)
- # clojure-norway (2)
- # clojure-spec (2)
- # clojure-uk (12)
- # clojurescript (3)
- # clojurewerkz (1)
- # core-async (21)
- # cursive (9)
- # datomic (37)
- # duct (3)
- # emacs (16)
- # events (4)
- # fulcro (34)
- # graalvm (12)
- # javascript (3)
- # jobs (4)
- # malli (1)
- # meander (3)
- # nrepl (1)
- # off-topic (27)
- # pathom (16)
- # re-frame (17)
- # reagent (19)
- # rewrite-clj (18)
- # sci (47)
- # shadow-cljs (179)
- # spacemacs (18)
- # sql (52)
- # tools-deps (80)
- # vim (27)
- # vrac (1)
- # xtdb (9)
Ya, pretty much @snoe! https://github.com/lread/clojure-lsp/commit/658e9cab297f08aa60bc09707ebdb25b40960ca6
If you ignore the hacks I did due to the fact that rewrite-cljc is not up on clojars yet.
beauty! I actually vendor rewrite-clj so I'm gonna convert this weekend. Amazing work and dedication on this project, thank you!
https://github.com/snoe/clojure-lsp/pull/150 I'm gonna mull it over, are you waiting on clj-commons before going to clojars?
snoe: fyi, I've also vendored rewrite-clj, for 3 reasons: 1) I didn't want to deal with whitespace because I'm only linting (for you this is different, since you actually rewrite). 2) fixed bugs (namespaced maps, maybe some others) 3) I wanted to deal differently with metadata nodes: I wanted metadata to be a child of a node, not a parent.
however, I might include the "real" rewrite-clj as well in the future, if actually want to rewrite something like clj-kondo config. who knows
@snoe, rewrite-cljc also brings in the paredit API from rewrite-cljs which might be of interest to you, dunno. https://github.com/lread/rewrite-cljc-playground/blob/master/src/rewrite_cljc/paredit.cljc
@slimslenderslacks, I resolved https://github.com/lread/rewrite-cljc-playground/commit/4c22974df4b0cde77a92e0eede4488be28da5e33 afterwhich https://github.com/lread/update-leiningen-dependencies-skill/commit/405d1783afed47d730712bbf93213d29af2ab897. Very glad to have had the opportunity to find and fix that Shadow CLJS issue!
@lee awesome, good to hear! That repo contains some code to watch clojars and send PRs to Repos that might need project.clj updates. We're about to release that to the community, along with a few other tools that people will hopefully find useful. One for formatting https://go.atomist.com/catalog/skills/atomist/cljfmt-skill and one for adding clj-kondo https://go.atomist.com/catalog/skills/atomist/clj-kondo-skill are almost ready to go (speaking of rewrite-clj!)
I'm excited to start using rewrite-cljc when it's ready