Fork me on GitHub
#nrepl
<
2019-11-10
>
dominicm08:11:04

There isn't booleans in bencode

dominicm08:11:18

It's been worked around mostly

pez09:11:51

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

pez09:11:40

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.

shen19:11:04

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

shen19:11:49

including the upgrading socket repl to nREPL one...

pez20:11:21

Oh, wow. Sounds crazily cool!

pez20:11:47

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

shen07:11:41

hmm.. how do you normally start the repl process?

shen07:11:44

lein repl?

shen07:11:17

think setting it in .nrepl.edn also works?

pez07:11:04

Calva supports tools.deps, lein and shadow-cljs. Ideally I’d like to add whatever is needed to the command lines starting the processes. But I’ll check out .nrep.edn too.

shen07:11:13

There was a discussion of configuring nrepl via JVM opts: https://github.com/nrepl/nrepl/issues/54

shen07:11:29

in lein, it's passed via :repl-options

shen07:11:24

guess .nrepl.edn is something different, but saves you the effort of threading config through whichever tool you are supporting?

pez08:11:19

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.

pez08:11:43

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

shen09:11:57

or rather, it seems to support .nrepl.edn

shen09:11:30

but not by adding things into shadow-cljs.edn

pez09:11:03

Thanks again!

pez09:11:56

@U05224H0W, 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.

thheller09:11:43

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

pez09:11:32

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.)

thheller09:11:29

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

thheller09:11:03

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

thheller09:11:39

but I guess I can add the config option regardless

pez09:11:41

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

shen09:11:41

I was just about to comment on that issue

thheller10:11:05

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

pez10:11:42

Working fast!