Fork me on GitHub
#cider
<
2019-01-11
>
bozhidar06:01:22

@borkdude What’s your nREPL version?

bozhidar06:01:59

It seems to work just fine for me.

bozhidar08:01:21

Note to everyone - early next week I plan to cut a new CIDER release, so if you wanted to get some features in, you don’t have much time.

benedek12:01:18

@kwcharllie379 i think this is your best shot nowadays https://nrepl.org/nrepl/usage/misc.html#_hot_loading_dependencies

manuel12:01:33

clj-refactor used to do it quite nicely

kwcharllie37912:01:16

@benedek is it works as well with java10?

borkdude12:01:05

@bozhidar

;; Connected to nREPL server - 
;; CIDER 0.20.0snapshot (package: 20190108.925), nREPL 0.5.3
;; Clojure 1.10.0, Java 1.8.0_191

benedek13:01:13

@kwcharllie379 this one should. the one in clj-refactor

benedek13:01:19

now disabled i think would not

kwcharllie37913:01:12

@bozhidar May I try to add a support for hot loading to clj-refactor via tools.deps or pomegranate?

benedek13:01:05

@kwcharllie379 i guess you could add the feature back by reimplementing it using the above (clj-refactor maintainer here 😉 )

kwcharllie37913:01:44

Thank you @benedek

benedek13:01:15

i thank you 🙂

bozhidar14:01:47

@kwcharllie379 Given how easy it is to support this now, as it’s a native thing in nREPL it can also be added straight to CIDER>

bozhidar14:01:58

This was also my intention, but I never got to do it.

bozhidar14:01:51

There’s also the point that hot-loading deps is not really a refactoring.

dominicm14:01:14

@kwcharllie379 the problem is that nREPL will need some shifting around to make the classloader available to middleware. That's probably a good change to make, I was admittedly hacking a bit when I slipped it in.

dominicm14:01:50

I haven't reviewed the latest work on nREPL yet, so that might have some impact.

dominicm14:01:26

@bozhidar I don't know how you find the energy 🙂 Since the new year started I'm just pooped.

bozhidar14:01:01

@dominicm You and me both. 😄 I just push myself to wrap completely the nREPL migration and starting next week I plan to take a vacation from all open-source.

dpsutton14:01:16

are you going to use that new github status feature?

dpsutton14:01:24

can't tell if people are liking it or not

bozhidar14:01:33

It’s ridiculous.

bozhidar14:01:49

I’ll just turn off my notifications and have people figure out stuff on their own .

bozhidar14:01:05

Frankly, even answering support questions here has become stressful for me.

dominicm06:01:32

I'm finding slack increasingly stressful. Zulip was great when people were interested, because it allowed me to balance my reading more carefully.

dpsutton14:01:11

that's what i have been hearing. just wanted to hear if anyone liked it 🙂

benedek14:01:15

good shout @bozhidar and @dominicm better in cider if possible

benedek14:01:16

i am working on a 0.5x branch of mranderson btw. well, as much as i can

benedek14:01:20

it is a clean up and rewrite-clj rewrite so far

bozhidar15:01:52

@benedek Fantastic news!

bozhidar15:01:51

@borkdude Everything seems good there. I don’t see any reason why the pprint won’t work.

bozhidar15:01:59

Guess you can check your nrepl log.

bozhidar15:01:08

Does pprint work normally in the REPL?

borkdude15:01:24

pprint works, but it seems to print with a normal pr-str, not with pprint

borkdude15:01:29

yes works in the REPL

bozhidar15:01:42

It’s the same code, so it’s highly unlikely it doesn’t work.

bozhidar15:01:55

What happens when you eval (range 100)?

borkdude15:01:08

you mean with cider-pprint?

bozhidar15:01:41

with C-c C-p

bozhidar15:01:57

You have to check your nrepl log and your pprint config then.

bozhidar15:01:05

This certainly provides different results for me.

borkdude15:01:34

(-->
  id            "45"
  op            "eval"
  session       "21325fbb-d5ad-49a1-98ee-85f205e44504"
  time-stamp    "2019-01-11 16:55:13.560653000"
  code          "(range 100)
"
  column        3
  file          "/Users/Borkdude/dre/DocSearch/app/src/dre/yada.clj"
  line          200
  ns            "dre.yada"
  print-options (dict
                  right-margin 80)
  printer       "cider.nrepl.pprint/pprint"
)

bozhidar15:01:02

Everything looks good.

bozhidar15:01:21

I can’t imagine how this wouldn’t work.

kwcharllie37916:01:31

I will try to figure it out then Ill ping you.