update sourcemaps integration

* switch to using built-in support in Gulp 4 for sourcemaps
* only enable when running preview unless SOURCEMAPS=true env var is set

diff --git a/gulp.d/tasks/build.js b/gulp.d/tasks/build.js
index 226a230..f9b4456 100644 (file)
--- a/gulp.d/tasks/build.js
+++ b/gulp.d/tasks/build.js
@@ -17,11 +17,11 @@ const postcssImport = require('postcss-import')
 const postcssUrl = require('postcss-url')
 const postcssVar = require('postcss-custom-properties')
 const uglify = require('gulp-uglify')
-const sourcemaps = require('gulp-sourcemaps')
 const vfs = require('vinyl-fs')
 
 module.exports = (src, dest, preview) => () => {
   const opts = { base: src, cwd: src }
+  const sourcemaps = preview || process.env.SOURCEMAPS === 'true'
   const postcssPlugins = [
     postcssImport(),
     postcssUrl([
@@ -45,11 +45,9 @@ module.exports = (src, dest, preview) => () => {
 
   return merge(
     vfs
-      .src('js/+([0-9])-*.js', opts)
-      .pipe(sourcemaps.init())
+      .src('js/+([0-9])-*.js', { ...opts, sourcemaps })
       .pipe(uglify())
-      .pipe(concat('js/site.js'))
-      .pipe(sourcemaps.write('.')),
+      .pipe(concat('js/site.js')),
     vfs
       .src('js/vendor/*.js', { ...opts, read: false })
       .pipe(
@@ -84,5 +82,5 @@ module.exports = (src, dest, preview) => () => {
     vfs.src('helpers/*.js', opts),
     vfs.src('layouts/*.hbs', opts),
     vfs.src('partials/*.hbs', opts)
-  ).pipe(vfs.dest(dest))
+  ).pipe(vfs.dest(dest, { sourcemaps: sourcemaps && '.' }))
 }

diff --git a/package.json b/package.json
index 8014891..0ba8142 100644 (file)
--- a/package.json
+++ b/package.json
@@ -29,7 +29,6 @@
     "gulp-eslint": "~5.0",
     "gulp-imagemin": "~5.0",
     "gulp-postcss": "~8.0",
-    "gulp-sourcemaps": "~2.6",
     "gulp-stylelint": "~8.0",
     "gulp-uglify": "~3.0",
     "gulp-vinyl-zip": "~2.1 >=2.1.2",