jupl / boot-cljs-devtools Goto Github PK

I have the following in my usual app.cljs and I was wondering if I can have the same when using boot-cljs-devtools:

(devtools.core/install! [:custom-formatters :sanity-hints])

It is basically adding :sanity-hints for devtools together with the default :custom-formatters.

Maybe we can add an option?

My pipeline is surrounded by a watch and reload and I see many times the string:

Adding :require [devtools.core dirac.runtime] and :init-fns [devtools.core/install! dirac.runtime/install!] to main.cljs.edn...

Is there a way to detect if dirac/devtools is installed already and skip it? I will ask on Slack, just opened this for reminding it to myself 😄

I see that at the moment there is a warning when either cljs-devtools or dirac is missing in the env.

I have in lambone an optional switch to enable dirac though which at the moment conditionally includes the deps.

It would be great if boot-cljs-devtools could detect that I don't have one or the other in the env and instead of send back an error, just don't require the namespace and don't call the install! on that missing dep.

What do you all think?

I have the following pipeline:

(comp (apply serve (mapcat identity (:serve options)))
             (built-in/watch)
             (apply sass (mapcat identity (:sass options)))
             (apply reload (mapcat identity (:reload options)))
             (apply cljs-devtools (mapcat identity cljs-devtools))
(built-in/show :fileset true) <- added
             (apply cljs (mapcat identity (:cljs options))))

And apart from the apply noise you can see that I am showing the fileset between the tasks.

Well, I have no .cljs.edn file (as I am using straight main.js for my built artifact.

So is there a more reliable way in your opinion to add this code?

It would be better to let the user specify them explicitly in build.boot or we can include them conditionally if they are not already in the (get-env).