search

crisptrutski / boot-cljs-test

Boot task to run ClojureScript tests.
53 stars 18 forks source link

Support new boot-cljs `2.1.2` #65

Closed burn2delete closed 7 years ago

burn2delete commented 7 years ago

This error is happening as of upgrading from boot-cljs 2.0.0

crisptrutski commented 7 years ago

Thanks for reporting - do you have a stacktrace?

crisptrutski commented 7 years ago

Looks like something to do with the asset-path changes - are you also getting this: ClojureScript could not load :main, did you forget to specify :asset-path??

crisptrutski commented 7 years ago

Dug into a few cases and can confirm the issue is with the changed asset-path convention - eg. with :id beep/boop the stub index.html is being created inside the beep directory, and then looking for beep/boop.out/goog/base.js. i.e. the path is being handled twice.

Will likely need to cut a new version with either no prior to 2.1.0 boo-cljs support, or branching on the version found. Should be able to find some time this weekend 🙂

burn2delete commented 7 years ago

Yes that is the correct error, I'm not sure why I didn't paste it before.

crisptrutski commented 7 years ago

0.3.3-SNAPSHOT published for testing 🌮

burn2delete commented 7 years ago

working!

crisptrutski commented 7 years ago

Thanks for confirming 😌