nrepl

https://github.com/nrepl/nrepl || https://nrepl.org
dominicm 2019-11-10T08:42:04.025700Z

There isn't booleans in bencode

dominicm 2019-11-10T08:42:18.026100Z

It's been worked around mostly

pez 2019-11-10T09:15:51.027200Z

Yeah, @kkinnear is going to work around it for me. 😎

pez 2019-11-10T09:45:40.029100Z

And I am starting to think I should try use the EDN transport instead. A lot of the problems I have fought lately has been related to that I do not allow the data to stay in EDN as far as possible.

pez 2019-11-11T08:29:19.032600Z

Thanks. I was looking at the shadow-docs yesterday and it wasn't obvious to me what I should put there to get the EDN selected.

pez 2019-11-11T08:29:43.032800Z

(Still isn't. so if you can give me a hint. 😃 )

shen 2019-11-11T09:03:05.033Z

I'm looking now, think shadow might need to do some work to support it: https://github.com/thheller/shadow-cljs/blob/92bb8076f6d1774bdbcf32dc8b0e14ce949c769a/src/main/shadow/cljs/devtools/server/nrepl.clj#L123

shen 2019-11-11T09:03:57.033300Z

or rather, it seems to support .nrepl.edn

shen 2019-11-11T09:04:30.033500Z

but not by adding things into shadow-cljs.edn

pez 2019-11-11T09:19:03.033700Z

Thanks again!

pez 2019-11-11T09:21:56.033900Z

@thheller, is this something you would consider adding as a configuration option? I could probably go via .nrepl.edn, and maybe even should go that way, but so far I haven't let Calva edit config files on the user's behalf. It's pretty nice to be able to specify these things on the command line.

thheller 2019-11-11T09:25:43.034100Z

add what? not entirely sure what you are talking about here

pez 2019-11-11T09:35:32.034300Z

Sorry. Add a way to config the nrepl server to use another transport. (EDN transport is what I currently am considering to try with for Calva.)

thheller 2019-11-11T09:36:29.034500Z

as soon as the alpha tag disappears I guess? https://github.com/thheller/shadow-cljs/issues/514

thheller 2019-11-11T09:37:03.034800Z

don't know why it has an alpha tag, not keeping up with whats going on over there

thheller 2019-11-11T09:37:39.035Z

but I guess I can add the config option regardless

1🤘
pez 2019-11-11T09:37:41.035200Z

The API is not settled, is one thing I have picked up.

shen 2019-11-11T09:52:41.035500Z

I was just about to comment on that issue

thheller 2019-11-11T10:00:05.035900Z

trying to finish up some other work. will probably make a release later

pez 2019-11-11T10:11:42.036900Z

Working fast!

shen 2019-11-10T19:37:47.029200Z

I wonder if we should read-string the value?

shen 2019-11-10T19:39:39.029500Z

It feels quite limiting to only be able to use what can survive the bencode conversion to and fro

shen 2019-11-10T19:43:23.029700Z

Googling ":nrepl.middleware.print/options" doesn't yield that many results, so I'm guessing this doesn't get used much other than to pass {:right-margin 70} or similar around

shen 2019-11-10T19:49:04.029900Z

There's also a few features on the horizon that might be EDN transport only.

shen 2019-11-10T19:49:49.030100Z

including the upgrading socket repl to nREPL one...

pez 2019-11-10T20:25:21.030300Z

Oh, wow. Sounds crazily cool!

pez 2019-11-10T20:26:47.030500Z

Is it still only clojure cli where I can start the edn transport server via a command line switch?