Skip to content

Commit 72ba408

Browse files
authored
Merge pull request #147 from webpack-contrib/d3viant0ne-TravisUpdates
fix(getOptions): deprecation warn in loaderUtils
2 parents dd369b0 + 30b9e1e commit 72ba408

File tree

6 files changed

+2234
-14
lines changed

6 files changed

+2234
-14
lines changed

.gitattributes

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
yarn.lock -diff

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
11
/node_modules
2-
test/output/*
2+
test/node_modules/*
3+
test/output/*

.travis.yml

Lines changed: 33 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,36 @@
1+
# Tasks are commented out purposfully. Will be uncommented for webpack-defaults application
12
sudo: false
23
language: node_js
3-
node_js:
4-
- "0.10"
5-
- "0.11"
6-
- "0.12"
7-
- "4"
4+
branches:
5+
only:
6+
- master
7+
matrix:
8+
fast_finish: true
9+
include:
10+
- os: linux
11+
node_js: '6'
12+
env: JOB_PART=test
13+
- os: linux
14+
node_js: '4.3'
15+
env: JOB_PART=test
16+
- os: linux
17+
node_js: '7'
18+
env: JOB_PART=test
19+
# - os: linux
20+
# node_js: '7'
21+
# env: WEBPACK_VERSION="2.2.0" BITHOUND_CHECK=true JOB_PART=lint
22+
# - os: linux
23+
# node_js: '7'
24+
# env: WEBPACK_VERSION="2.2.0" JOB_PART=coverage
25+
before_install:
26+
- nvm --version
27+
- node --version
28+
before_script:
29+
# - if [ "$WEBPACK_VERSION" ]; then yarn add webpack@^$WEBPACK_VERSION; fi
30+
# - if [ "$BITHOUND_CHECK" ]; then npm install -g bithound; bithound check [email protected]:$TRAVIS_REPO_SLUG.git; fi
831
script:
9-
- "npm test"
10-
# test with npm@3
11-
- "rm -rf node_modules && npm install npm@3"
12-
- "node_modules/.bin/npm install"
13-
- "node_modules/.bin/npm test"
32+
- yarn run travis:$JOB_PART
33+
# after_success:
34+
# - bash <(curl -s https://codecov.io/bash)
35+
36+
# TODO: This test suite will throw schema errors with webpack 2.2.x installed. Address this on defaults upgrade to Jest

index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ var trailingSlash = /[\\\/]$/;
1515

1616
module.exports = function(source) {
1717
var loaderContext = this;
18-
var query = loaderUtils.parseQuery(this.query);
18+
var query = loaderUtils.getOptions(this) || {};
1919
var cb = this.async();
2020
var isSync = typeof cb !== "function";
2121
var finalCb = cb || this.callback;

package.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@
44
"author": "Tobias Koppers @sokra",
55
"description": "less loader module for webpack",
66
"scripts": {
7-
"test": "node --no-deprecation node_modules/.bin/_mocha -R spec",
7+
"test": "npm run travis:test",
8+
"travis:test": "node --no-deprecation node_modules/.bin/_mocha -R spec",
89
"test-source-map": "webpack --config test/sourceMap/webpack.config.js && open ./test/sourceMap/index.html"
910
},
1011
"peerDependencies": {
@@ -31,6 +32,6 @@
3132
}
3233
],
3334
"dependencies": {
34-
"loader-utils": "^0.2.5"
35+
"loader-utils": "^1.0.2"
3536
}
3637
}

0 commit comments

Comments
 (0)