Merge pull request #4 from bchiang7/audit

Update a few packages

build/browsersync.js

@@ -1,23 +1,30 @@
-const browserSync  = require('browser-sync').create();
+const browserSync = require('browser-sync').create();
-const cp           = require('child_process');
+const cp = require('child_process');
 
-const jekyll       = process.platform === 'win32' ? 'jekyll.bat' : 'jekyll';
+const jekyll = process.platform === 'win32' ? 'jekyll.bat' : 'jekyll';
 
-const scssPath     = '_scss/**/*.scss';
+const scssPath = '_scss/**/*.scss';
-const jsPath       = '_scripts/*.js';
+const jsPath = '_scripts/*.js';
-const templatePath = ['*.html', '+(_includes|_layouts)/*.html', '*.yml', '_data/*.yml', '_posts/*'];
+const templatePath = [
+  '*.html',
+  '+(_includes|_layouts)/*.html',
+  '*.yml',
+  '_data/*.yml',
+  '_posts/*',
+];
 
 module.exports = gulp => {
-
   // run `jekyll build`
   gulp.task('jekyll-build', done => {
-    return cp.spawn(jekyll, ['build'], {stdio: 'inherit'})
+    return cp.spawn(jekyll, ['build'], { stdio: 'inherit' }).on('close', done);
-      .on('close', done);
   });
 
   // run `jekyll build` with _config_dev.yml
   gulp.task('jekyll-dev', done => {
-    return cp.spawn(jekyll, ['build', '--config', '_config.yml,_config_dev.yml'], {stdio: 'inherit'})
+    return cp
+      .spawn(jekyll, ['build', '--config', '_config.yml,_config_dev.yml'], {
+        stdio: 'inherit',
+      })
       .on('close', done);
   });
 
@@ -29,13 +36,12 @@ module.exports = gulp => {
   gulp.task('serve', ['jekyll-dev'], () => {
     browserSync.init({
       server: {
-        baseDir: '_site'
+        baseDir: '_site',
-      }
+      },
     });
 
     gulp.watch(scssPath, ['sass', browserSync.reload]);
     gulp.watch(jsPath, ['scripts', browserSync.reload]);
     gulp.watch(templatePath, ['jekyll-rebuild']);
   });
-
 };

build/images.js

@@ -4,11 +4,10 @@ const imgPath = 'img/**/*.+(png|jpg|gif|svg)';
 const destPath = '_site/img';
 
 module.exports = gulp => {
-
   gulp.task('images', () => {
-    return gulp.src(imgPath)
+    return gulp
+      .src(imgPath)
       .pipe(imagemin())
       .pipe(gulp.dest(destPath));
   });
-
 };

build/sass.js

@@ -1,25 +1,28 @@
-const sass        = require('gulp-sass');
+const sass = require('gulp-sass');
-const prefix      = require('gulp-autoprefixer');
+const prefix = require('gulp-autoprefixer');
-const cleanCSS    = require('gulp-clean-css');
+const cleanCSS = require('gulp-clean-css');
 
-const scssPath    = '_scss/*.scss';
+const scssPath = '_scss/*.scss';
-const destPath    = '_site/css';
+const destPath = '_site/css';
 
 module.exports = gulp => {
-
   gulp.task('sass', () => {
-    return gulp.src(scssPath)
+    return gulp
-      .pipe(sass({
+      .src(scssPath)
-        includePaths: ['scss'],
+      .pipe(
-        outputStyle: 'expanded'
+        sass({
-      }))
+          includePaths: ['scss'],
-      .pipe(prefix({
+          outputStyle: 'expanded',
-        browsers: ['last 2 versions'],
+        })
-        cascade: false
+      )
-      }))
+      .pipe(
+        prefix({
+          browsers: ['last 2 versions'],
+          cascade: false,
+        })
+      )
       .pipe(cleanCSS({ compatibility: 'ie8' }))
       .pipe(gulp.dest(destPath))
       .pipe(gulp.dest('css'));
   });
-
 };

build/scripts.js

@@ -1,20 +1,23 @@
-const eslint   = require('gulp-eslint');
+const eslint = require('gulp-eslint');
-const uglify   = require('gulp-uglify');
+const uglify = require('gulp-uglify');
 
-const jsPath   = '_scripts/*.js';
+const jsPath = '_scripts/*.js';
 const destPath = '_site/js';
 
 module.exports = gulp => {
-
   gulp.task('scripts', () => {
-    return gulp.src(jsPath)
+    return (
-      .pipe(eslint({
+      gulp
-        useEslintrc: true
+        .src(jsPath)
-      }))
+        .pipe(
-      .pipe(eslint.format())
+          eslint({
-      // .pipe(uglify())
+            useEslintrc: true,
-      .pipe(gulp.dest(destPath))
+          })
-      .pipe(gulp.dest('js'));
+        )
+        .pipe(eslint.format())
+        // .pipe(uglify())
+        .pipe(gulp.dest(destPath))
+        .pipe(gulp.dest('js'))
+    );
   });
-
 };

gulpfile.js

@@ -1,10 +1,8 @@
-'use strict';
+const gulp = require('gulp');
-
+const sass = require('./build/sass');
-const gulp    = require('gulp');
-const sass    = require('./build/sass');
 const scripts = require('./build/scripts');
-const images  = require('./build/images');
+const images = require('./build/images');
-const sync    = require('./build/browsersync');
+const sync = require('./build/browsersync');
 
 [sass, scripts, images, sync].forEach(task => {
   task(gulp);

package.json

@@ -11,14 +11,14 @@
   "license": "ISC",
   "devDependencies": {
     "@nuscout/eslint-config": "^0.2.1",
-    "browser-sync": "^2.24.5",
+    "browser-sync": "^2.24.7",
     "gulp": "3.9.1",
-    "gulp-autoprefixer": "^5.0.0",
+    "gulp-autoprefixer": "^6.0.0",
-    "gulp-clean-css": "3.9.4",
+    "gulp-clean-css": "3.10.0",
     "gulp-eslint": "5.0.0",
     "gulp-imagemin": "4.1.0",
     "gulp-sass": "^4.0.1",
-    "gulp-uglify": "3.0.0"
+    "gulp-uglify": "3.0.1"
   },
   "repository": {
     "type": "git",