Skip to content

Commit 770722b

Browse files
authored
Merge pull request #383 from jtangelder/update/dependencies
Update/dependencies
2 parents 42bfebb + 47cead4 commit 770722b

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

lib/normalizeOptions.js

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
const os = require("os");
44
const utils = require("loader-utils");
5+
const cloneDeep = require("clone-deep");
56
const path = require("path");
67
const proxyCustomImporters = require("./proxyCustomImporters");
78

@@ -17,12 +18,7 @@ const proxyCustomImporters = require("./proxyCustomImporters");
1718
* @returns {Object}
1819
*/
1920
function normalizeOptions(loaderContext, content, webpackImporter) {
20-
const options = loaderContext.query && typeof loaderContext.query === "object" ?
21-
// Make a copy of the query object
22-
// @see https://github.com/jtangelder/sass-loader/issues/368#issuecomment-278330164
23-
Object.assign({}, loaderContext.query) :
24-
utils.parseQuery(loaderContext.query);
25-
21+
const options = cloneDeep(utils.getOptions(loaderContext)) || {};
2622
const resourcePath = loaderContext.resourcePath;
2723

2824
options.data = options.data ? (options.data + os.EOL + content) : content;
@@ -73,7 +69,7 @@ function normalizeOptions(loaderContext, content, webpackImporter) {
7369
options.importer.push(webpackImporter);
7470

7571
// `node-sass` uses `includePaths` to resolve `@import` paths. Append the currently processed file.
76-
options.includePaths = options.includePaths ? [].concat(options.includePaths) : [];
72+
options.includePaths = options.includePaths || [];
7773
options.includePaths.push(path.dirname(resourcePath));
7874

7975
return options;

package.json

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,16 @@
3131
"webpack": "^2.0.0"
3232
},
3333
"dependencies": {
34-
"async": "^2.0.1",
35-
"loader-utils": "^0.2.15",
34+
"async": "^2.1.5",
35+
"clone-deep": "^0.2.4",
36+
"loader-utils": "^1.0.1",
3637
"lodash.tail": "^4.1.1",
3738
"pify": "^2.3.0"
3839
},
3940
"devDependencies": {
4041
"bootstrap-sass": "^3.3.5",
4142
"css-loader": "^0.26.1",
42-
"eslint": "^3.15.0",
43+
"eslint": "^3.16.0",
4344
"eslint-config-peerigon": "^9.0.0",
4445
"eslint-plugin-jsdoc": "^2.4.0",
4546
"file-loader": "^0.10.0",
@@ -49,7 +50,7 @@
4950
"should": "^11.2.0",
5051
"style-loader": "^0.13.1",
5152
"webpack": "^2.2.1",
52-
"webpack-dev-server": "^2.3.0",
53-
"webpack-merge": "^2.6.1"
53+
"webpack-dev-server": "^2.4.1",
54+
"webpack-merge": "^3.0.0"
5455
}
5556
}

0 commit comments

Comments
 (0)