https://technomancy.us/186

has anyone got fireplace working with shadow-cljs?

i have it working fine with figwheel, but when i try and connect to a runnign shadow-cljs repl i get compileexceptions

supposedly you can run this: (shadow.cljs.devtools.api/nrepl-select :id-of-build)

@christopher.paul don't do that, do :Piggieback :id-of-build

ah

that’s all i hae to do? @dominicm?

Yep 🙂

normally my fireplace connects automatically

but with the shadow one

it doesn’t seem to…

I'm not certain if shadow writes the .nrepl-port, I've never checked.

i’ve manually created a file onw

for .nrepl-port

maybe it will just work…

You can also use :Connect @christopher.paul

i know, but i’m lazy 🙂

i’ve added the .nrepl_port maybe it is connecting and i didn’t realise

omg

it works

!!!!

@christopher.paul I worked with thheller to get it working a little while ago 🙂

shadow-cljs writes a file with the port inside target/, someone should make a pr on fireplace to use that

yea i saw you guys talking on one of the github issues

you have just made me a very happy koala

i used the shadow lein template, these .edn files seem much nicer than a project.clj

@mynomoto eh, that seems inappropriate for A) how fireplace works B) for shadow to violate how other nrepl things work. I'm sure there's a good reason of course, but it seems a strange thing to add to fireplace. Not sure.

well i think fireplace is already looking for one file, i think mynomoto is suggesting to try looking in a second place…

or maybe shadow could just write to the same place as lein and boot 😄

fireplace looks recursively upwards for .nrepl-port files, and scopes the connection appropriately.

maybe i’ll create a command in my vim rc so i don’t have to manually create .nrepl_port files

@christopher.paul thomas said that he wants to avoid conflicts with files written by other tooling as there is reason to run both lein or boot and shadow in the same project.

ok

@dominicm shadow uses target/shadow-cljs/nrepl.port, so that won't work as it is.

thanks for your help, @dominicm

@mynomoto Makes sense, I had thought it might be like that.