Skip to content
This repository was archived by the owner on Nov 4, 2021. It is now read-only.

Commit df77c62

Browse files
committed
(fix): use lodash.merge instead of lodash.assign
Using `lodash.assign` overwrites the default values when updating nested options.
1 parent adb1785 commit df77c62

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

sass-build/index.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
var gulp = require('gulp'),
22
sass = require('gulp-sass'),
3-
autoprefixer = require('gulp-autoprefixer')
4-
assign = require('lodash.assign');
3+
autoprefixer = require('gulp-autoprefixer'),
4+
merge = require('lodash.merge');
55

66
var defaultOptions = {
77
src: 'app/theme/app.+(ios|md|wp).scss',
@@ -29,7 +29,7 @@ var defaultOptions = {
2929
}
3030

3131
module.exports = function(options) {
32-
options = assign(defaultOptions, options);
32+
options = merge(defaultOptions, options);
3333

3434
return gulp.src(options.src)
3535
.pipe(sass(options.sassOptions))

0 commit comments

Comments
 (0)