hoplon + boot-figreload generates invalid edn about boot-figreload

[arichiardi]

Thanks for the report, yes that does not look right at all. I am not familiar with hoplon at all but if you have a minimal repro repo, or want to use a branch in this repo I can have a look.

Also I see you haven't posted any .cljs.edn file here, just want to point out that it is mandatory for boot-figreload (and in general is good to have it for that matter).

[freakinruben]

I'm not super familiar with Hoplon myself, but here is the repo cloned and adjusted (to my best knowledge) for hoplon. Weather or not the hoplon part is correct, running boot dev will give the same error as I have with my own code.

About the .cljs.edn file, hoplon generates this for you. The generated edn before figreload with the repo above is:

{:require [figreload-demo.index adzerk.boot-cljs-repl]}

After:

{:require [figreload-demo.index figwheel.connect./Users/foo/.boot/cache/tmp/Users/foo/Projects/figreload-demo/6h7/-7bl902/index.html.cljs.edn-72fbe00c adzerk.boot-cljs-repl]}

[arichiardi]

Ok that is great thanks, I will have a look.

[arichiardi]

It looks like hoplon is creating a .cljs.edn file that fails my regex here...I don't have much time now, but a patch should be really trivial, I am basically taking the build-id, the part before cljs.edn which in your case above is index.html but really should be index.

[vigilancetech-com]

I replaced the (\w+) with ([a-zA-Z0-9_.]+) and even tho at the REPL, when given "index.html.cljs.edn" it returns "index.html" it still give the error.

I also changed it to ([a-zA-Z0-9_]+).(.+) which only returns "index" and no improvement.

I'd like to start using this ASAP. Any more ideas?

[arichiardi]

@vigilancetech-com what error do you get? Same as above? It's been a while but I though the regex would solve.