crisptrutski / boot-cljs-test

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

Support -vv style verbosity control #39

Closed crisptrutski closed 7 years ago

crisptrutski commented 8 years ago

Doo takes two output options: debug and verbose.

The most naturally-boot way to handle this would be via a "stacking" log level argument:

boot cljs-test                verbose=false, debug=false
boot cljs-test -v             verbose=true,  debug=false
boot cljs-test -vv            verbose=true,  debug=true
Deraen commented 8 years ago

Boot already has global verbose flag. It sets dynamic var boot.util/*verbosity*: https://github.com/boot-clj/boot/blob/master/boot/pod/src/boot/util.clj#L31-L43

The value can be used to set options for Doo.

crisptrutski commented 8 years ago

Is this the accepted practice? Only concern is that user may only be wanting "extreme cljs-test detail", and they'd need to look through the noise from other tasks, and lower level boot constructs

crisptrutski commented 7 years ago

Adding number flag, ie. -v 2 to set log level. But if that's not set, it'll default to the *verbosity* var.