fix(cli): can explicitly set the log level to “default” for the Listr renderer

Fixes #154

README.md

@@ -136,7 +136,7 @@ Website options:
 
 Misc options:
   --config       JSON config file (one key per argument)  [string]
-  --log          Print a detailed text log  [choices: null, "info", "debug", "trace"] [default: null]
+  --log          Print a detailed text log  [choices: "default", "info", "debug", "trace"] [default: "default"]
   --usage-stats  Enable anonymous usage statistics  [boolean] [default: true]
   --dry-run      Update the index, but don't create the media files / website  [boolean] [default: false]
 

bin/log.js

@@ -27,12 +27,12 @@ const tty = require('tty')
 exports.init = (logLevel, outputFolder) => {
   // if the output doesn't support ANSI codes (e.g. pipe, redirect to file)
   // then switch to full-text mode, because Listr's output won't make much sense
-  if (!logLevel && !tty.isatty(process.stdout.fd)) {
+  if (logLevel === 'default' && !tty.isatty(process.stdout.fd)) {
     logLevel = 'info'
   }
 
   // Configure the loggers
-  if (!logLevel) {
+  if (logLevel === 'default') {
     configureDefaultMode(outputFolder)
   } else {
     configureDebugMode(logLevel)

bin/options.js

@@ -271,8 +271,8 @@ const OPTIONS = {
   'log': {
     group: 'Misc options:',
     description: 'Print a detailed text log',
-    choices: [null, 'info', 'debug', 'trace'],
-    'default': null
+    choices: ['default', 'info', 'debug', 'trace'],
+    'default': 'default'
   },
 
   'usage-stats': {

src/index.js

@@ -5,7 +5,7 @@ const Problems = require('./problems')
 
 exports.build = function (opts, done) {
   // How to render tasks
-  const renderer = (!opts.log) ? 'update' : 'verbose'
+  const renderer = (opts.log === 'default') ? 'update' : 'verbose'
   // List of high level tasks
   const tasks = new Listr([
     {