diff --git a/lib/versioned/^4.0.0-alpha.1/index.js b/lib/versioned/^4.0.0-alpha.1/index.js index 5660783b..4feb4163 100644 --- a/lib/versioned/^4.0.0-alpha.1/index.js +++ b/lib/versioned/^4.0.0-alpha.1/index.js @@ -51,9 +51,8 @@ function execute(opts, env) { var output = JSON.stringify(gulpInst.tree({ deep: true })); if (typeof opts.tasksJson === 'boolean' && opts.tasksJson) { return console.log(output); - } else { - return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } + return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } try { log.info('Using gulpfile', chalk.magenta(tildify(env.configPath))); diff --git a/lib/versioned/^4.0.0-alpha.2/index.js b/lib/versioned/^4.0.0-alpha.2/index.js index be0f6eea..c59f1bdf 100644 --- a/lib/versioned/^4.0.0-alpha.2/index.js +++ b/lib/versioned/^4.0.0-alpha.2/index.js @@ -57,9 +57,8 @@ function execute(opts, env) { if (typeof opts.tasksJson === 'boolean' && opts.tasksJson) { return console.log(output); - } else { - return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } + return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } try { log.info('Using gulpfile', chalk.magenta(tildify(env.configPath))); diff --git a/lib/versioned/^4.0.0/index.js b/lib/versioned/^4.0.0/index.js index 635665ec..782ecc42 100644 --- a/lib/versioned/^4.0.0/index.js +++ b/lib/versioned/^4.0.0/index.js @@ -57,9 +57,8 @@ function execute(opts, env) { if (typeof opts.tasksJson === 'boolean' && opts.tasksJson) { return console.log(output); - } else { - return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } + return fs.writeFileSync(opts.tasksJson, output, 'utf-8'); } try { log.info('Using gulpfile', chalk.magenta(tildify(env.configPath))); diff --git a/lib/versioned/^4.0.0/log/events.js b/lib/versioned/^4.0.0/log/events.js index 7a1b5fc8..6ee8c380 100644 --- a/lib/versioned/^4.0.0/log/events.js +++ b/lib/versioned/^4.0.0/log/events.js @@ -8,29 +8,36 @@ var formatError = require('../formatError'); // Wire up logging events function logEvents(gulpInst) { - gulpInst.on('start', function(e) { + var loggedErrors = []; + + gulpInst.on('start', function(evt) { // TODO: batch these // so when 5 tasks start at once it only logs one time with all 5 - log.info('Starting', '\'' + chalk.cyan(e.name) + '\'...'); + log.info('Starting', '\'' + chalk.cyan(evt.name) + '\'...'); }); - gulpInst.on('stop', function(e) { - var time = prettyTime(e.duration); + gulpInst.on('stop', function(evt) { + var time = prettyTime(evt.duration); log.info( - 'Finished', '\'' + chalk.cyan(e.name) + '\'', + 'Finished', '\'' + chalk.cyan(evt.name) + '\'', 'after', chalk.magenta(time) ); }); - gulpInst.on('error', function(e) { - var msg = formatError(e); - var time = prettyTime(e.duration); + gulpInst.on('error', function(evt) { + var msg = formatError(evt); + var time = prettyTime(evt.duration); log.error( - '\'' + chalk.cyan(e.name) + '\'', + '\'' + chalk.cyan(evt.name) + '\'', chalk.red('errored after'), chalk.magenta(time) ); - log.error(msg); + + // If we haven't logged this before, log it and add to list + if (loggedErrors.indexOf(evt.error) === -1) { + log.error(msg); + loggedErrors.push(evt.error); + } }); }