Update build task

tasks/build.js

@@ -1,6 +1,6 @@
 'use strict';
 
-var gulp = require('gulp'),
+const gulp = require('gulp'),
   inlineCss = require('gulp-inline-css'),
   minifyHTML = require('gulp-minify-html'),
   minifyInline = require('gulp-minify-inline'),
@@ -14,15 +14,18 @@ var gulp = require('gulp'),
   inlineimg = require('gulp-inline-images-no-http'),
   path = require('path');
 
-function buildTask(options){
-  gulp.task('build', ['dupe', 'less', 'sass', 'postcss', 'lint'], function build() {
+function buildTask(options) {
+  // Requires: 'dupe', 'less', 'sass', 'postcss', 'lint'.
+  gulp.task(
+    'build',
+    function build(done) {
       /** Makes templates for a given directory & its configurations.
        * @function makeTemplates
        * @param {String} dir Directory to make templates from.
        * @param {Array} confItems A list of configurations objects (usually persons) to make templates from.
        */
-    function makeTemplates(dir, confItems){
-      confItems.forEach(function handleConf(conf){
+      function makeTemplates(dir, confItems) {
+        confItems.forEach(function handleConf(conf) {
           var cwd = options.workingDir + '/' + dir;
           var stylesheets = [];
 
@@ -34,7 +37,7 @@ function buildTask(options){
             .on('readable', function walkTemplateDir() {
               var stylesheet;
 
-            while (stylesheet = this.read()) {
+              while ((stylesheet = this.read())) {
                 var relativePath = __dirname.substring(0, __dirname.lastIndexOf('/')) + '/tmp/' + dir;
                 stylesheets.push(stylesheet.path.replace(relativePath, ''));
               }
@@ -43,40 +46,46 @@ function buildTask(options){
               conf.stylesheets = stylesheets
                 .filter(function filterFiles(file) {
                   /* Read only CSS files. */
-                return (file.match(/.*\.css/)) ? true : false;
+                  return file.match(/.*\.css/) ? true : false;
                 })
                 .reduce(function(prev, current, index, acc) {
-                var cssPath = path.win32.basename(current)
-                return prev += '<link rel="stylesheet" href="' + cssPath + '">';
+                  var cssPath = path.win32.basename(current);
+                  return (prev += '<link rel="stylesheet" href="' + cssPath + '">');
                 }, '');
 
               options
                 .src([cwd + '/**/*.html', '!' + cwd + '/**/*.inc.html'])
-              .pipe(preprocess({
+                .pipe(
+                  preprocess({
                     context: conf
-              }))
+                  })
+                )
                 .pipe(inlineimg())
-              .pipe(inlineCss({
+                .pipe(
+                  inlineCss({
                     applyTableAttributes: true,
                     applyWidthAttributes: true,
                     preserveMediaQueries: true,
                     removeStyleTags: false
-              }))
-              .pipe(minifyHTML({quotes: true}))
+                  })
+                )
+                .pipe(minifyHTML({ quotes: true }))
                 .pipe(minifyInline())
-              .pipe(rename(function rename(path){
+                .pipe(
+                  rename(function rename(path) {
                     path.dirname = dir;
                     path.basename += '-' + conf.id;
                     return path;
-              }))
-              .pipe(gulp.dest(options.dist));
                   })
+                )
+                .pipe(gulp.dest(options.dist));
+            });
         });
       }
 
       /** Clean up & then read 'src' to generate templates (build entry point). */
       del(options.dist)
-      .then(function buildStart(){
+        .then(function buildStart() {
           /**
            * Loop through dirs and load their conf files.
            * Promisify all 'makeTemplate' calls and when resolved, make a call to the task (`cb`) to let gulp know we're done.
@@ -84,9 +93,7 @@ function buildTask(options){
           var files = [];
           var promises = [];
 
-        fs
-          .readdirSync('./' + options.workingDir)
-          .forEach(function readConfigurations(dir){
+          fs.readdirSync('./' + options.workingDir).forEach(function readConfigurations(dir) {
             /** NB: For 'watch' to properly work, the cache needs to be deleted before each require. */
             var confPath = '../tmp/' + dir + '/conf.json';
             var current = null;
@@ -95,9 +102,9 @@ function buildTask(options){
             delete require.cache[require.resolve(confPath)];
             current = require(confPath);
             if (current && current.length) {
-              confItems = [...current]
+              confItems = [...current];
             } else {
-              confItems = [current]
+              confItems = [current];
             }
 
             promises.push(makeTemplates(dir, confItems));
@@ -105,8 +112,10 @@ function buildTask(options){
 
           Q.all(promises);
         })
-      .catch(function (err) { console.log(err) })
-  });
+        .then(() => done())
+        .catch((err) => console.log(err));
+    }
+  );
 }
 
 module.exports = buildTask;