This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2019-10-20
Channels
- # announcements (2)
- # babashka (1)
- # bangalore-clj (1)
- # beginners (5)
- # cider (16)
- # clojure (125)
- # clojure-spec (3)
- # clojure-uk (5)
- # clojurescript (27)
- # cursive (46)
- # data-science (3)
- # dirac (2)
- # emacs (11)
- # fulcro (2)
- # graphql (4)
- # luminus (2)
- # nrepl (1)
- # pathom (15)
- # re-frame (1)
- # reagent (52)
- # shadow-cljs (149)
- # sql (11)
- # tools-deps (11)
- # xtdb (14)
We should probably move this to CIDER, as I don’t see much reason for this to be in cljr
. It’s not really related to refactoring and I assume it’s not tied to anything complex in the other package.
I think I’d be down for all of the non-nrepl refactoring to move over. If they don’t need the middleware and are decently understandable
That has been the plan for a while now. 🙂 There’s no nREPL specific refactoring, but there are a lot of things that require an AST to work 9(e.g. rename-symbol
). We’ve agreed a long time ago to move everything that doesn’t require the AST to orchard and cider-nrepl, but no one had much time to work on this. The AST-powered commands will be revisited once we figure out a better way to do the analysis. @benedek is working on some ideas in this area.
Regarding AST, there is also the tree-sitter path. It was mentioned in some other channel. Thinking about how the AST situation is for Calva, it almost hurts thinking about all the ASTs that are built and thrown away all the time. I really should look at consolidating that…
There are a couple of projects like https://github.com/oakmac/tree-sitter-clojure, but they don’t seem particularly mature. So, even if we go in this direction there’s some ground work that needs to be done.
The real value of tree-sitter would be if it really gets universally adopted. For our needs it’s all the same what do we use as long as it gets the job done.
I’m pretty sure MS are not planning to adopt it. They have their own tech that they fancy for this.