search

crisptrutski / boot-cljs-test

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

Don't leak dependencies into later tasks (eg. aot, uberjar) #37

Open Deraen opened 8 years ago

Deraen commented 8 years ago

Currently boot-cljs-test adds dependency to doo directly using set-env!. This causes side-effects in case if cljs-test is being run as part of package task, the doo dependency would be added to resulting uberjar.

Simple workaround would be to add the dependency with :scope "test" but even then if would case side-effect on classpath.

Proper way would be to use pods to isolate this dependency.

crisptrutski commented 8 years ago

Remember having then removing pods from my first attempt, but can't recall why. Probably was just doing it wrong :smile: