Skip to content

Commit 61cb2c5

Browse files
bhollowaybholloway
authored
Merge pull request #199 from bholloway/feature/make-removeCR-default-on-windows
Make removeCR default on windows
2 parents 1cc721f + ff44105 commit 61cb2c5

15 files changed

+94
-91
lines changed

‎packages/resolve-url-loader/index.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@
44
*/
55
'use strict';
66

7-
var path = require('path'),
7+
var os = require('os'),
8+
path = require('path'),
89
fs = require('fs'),
910
util = require('util'),
1011
loaderUtils = require('loader-utils'),
@@ -77,7 +78,7 @@ function resolveUrlLoader(content, sourceMap) {
7778
sourceMap: loader.sourceMap,
7879
engine : 'postcss',
7980
silent : false,
80-
removeCR : false,
81+
removeCR : os.EOL.includes('\r'),
8182
root : false,
8283
debug : false,
8384
join : joinFn.defaultJoin

‎packages/resolve-url-loader/test/webpack5-sassloader8-sass4/package.json

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,18 +5,18 @@
55
"node": ">=10.0.0"
66
},
77
"dependencies": {
8-
"css-loader": "^4.2.0",
9-
"file-loader": "^6.0.0",
8+
"css-loader": "^5.2.4",
9+
"file-loader": "^6.2.0",
1010
"last-call-webpack-plugin": "^3.0.0",
11-
"mini-css-extract-plugin": "^0.9.0",
12-
"node-sass": "^4.14.0",
13-
"optimize-css-assets-webpack-plugin": "^5.0.3",
14-
"sass-loader": "^8.0.2",
15-
"webpack": "next",
16-
"webpack-cli": "^3.3.12"
11+
"mini-css-extract-plugin": "^1.5.1",
12+
"node-sass": "^5.0.0",
13+
"optimize-css-assets-webpack-plugin": "^5.0.4",
14+
"sass-loader": "^11.0.1",
15+
"webpack": "^5.36.1",
16+
"webpack-cli": "^4.6.0"
1717
},
1818
"scripts": {
19-
"webpack-d": "webpack --debug --output-pathinfo --mode=development",
20-
"webpack-p": "webpack -p --mode=production"
19+
"webpack-d": "webpack --no-color --output-pathinfo --mode=development",
20+
"webpack-p": "webpack --no-color --mode=production"
2121
}
2222
}

‎test/cases/adjacent-asset.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ module.exports = test(
205205
`,
206206
compose(
207207
onlyMeta('meta.engine == "rework" && meta.version.webpack >= 5'),
208-
assertCssAndSourceMapContent('main.cbda67410944acc1f957.css'),
208+
assertCssAndSourceMapContent('main.c389f1c4a95b262f2ab0.css'),
209209
outdent
210210
)`
211211
/src/feature/index.scss
@@ -239,8 +239,8 @@ module.exports = test(
239239
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
240240
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
241241
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
242-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.cbda67410944acc1f95
243-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 7.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
242+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.c389f1c4a95b262f2ab
243+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
244244
`,
245245
compose(
246246
onlyMeta('meta.engine == "postcss" && meta.version.webpack < 4'),
@@ -325,7 +325,7 @@ module.exports = test(
325325
`,
326326
compose(
327327
onlyMeta('meta.engine == "postcss" && meta.version.webpack >= 5'),
328-
assertCssAndSourceMapContent('main.da861c3ae0aaf2e698b0.css'),
328+
assertCssAndSourceMapContent('main.d8a71c4765a29b23f124.css'),
329329
outdent
330330
)`
331331
/src/feature/index.scss
@@ -361,8 +361,8 @@ module.exports = test(
361361
3:17 ░░░░░░░░░░░░░░░░;⏎ 9:17 ░░░░░░░░░░░░░░░░; }⏎
362362
}░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
363363
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
364-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.da861c3ae0aaf2e698
365-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ b0.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
364+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.d8a71c4765a29b23f1
365+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 24.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
366366
`
367367
),
368368
buildProdNormal(

‎test/cases/common/test/valid.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,16 +53,16 @@ exports.testDebug = (...elements) =>
5353
)
5454
);
5555

56-
exports.testRemoveCR = (...elements) =>
56+
exports.testRemoveCR = (removeCR) => (...elements) =>
5757
test(
58-
'removeCR=true',
58+
`removeCR=${removeCR}`,
5959
layer()(
6060
env({
61-
LOADER_OPTIONS: {removeCR: true},
62-
OUTPUT: 'remove-cr'
61+
LOADER_OPTIONS: {removeCR},
62+
OUTPUT: `remove-cr-${removeCR}`
6363
}),
6464
...elements,
65-
test('validate', assertStderr('options.removeCR')(1)`removeCR: true`)
65+
test('validate', assertStderr('options.removeCR')(1)`removeCR: ${String(removeCR)}`)
6666
)
6767
);
6868

‎test/cases/declaration-mixin.postcss.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ module.exports = test(
129129
`,
130130
compose(
131131
onlyMeta('meta.version.webpack >= 5'),
132-
assertCssAndSourceMapContent('main.9325b9f355d213ed8da0.css'),
132+
assertCssAndSourceMapContent('main.00abd28f6856af6b42c9.css'),
133133
outdent
134134
)`
135135
/src/index.scss
@@ -144,8 +144,8 @@ module.exports = test(
144144
2:35 ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░;⏎ 2:37 ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░; }⏎
145145
}░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
146146
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
147-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.9325b9f355d213ed8d
148-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ a0.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
147+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.00abd28f6856af6b42
148+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ c9.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
149149
`
150150
),
151151
buildProdNormal(
@@ -317,7 +317,7 @@ module.exports = test(
317317
`,
318318
compose(
319319
onlyMeta('meta.version.webpack >= 5'),
320-
assertCssAndSourceMapContent('main.4cd2bacae33ac5c9a2c4.css'),
320+
assertCssAndSourceMapContent('main.613c31543a6423df814a.css'),
321321
outdent
322322
)`
323323
/src/index.scss
@@ -332,8 +332,8 @@ module.exports = test(
332332
2:35 ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░;⏎ 2:45 ; }⏎
333333
}░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
334334
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
335-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.4cd2bacae33ac5c9a2
336-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ c4.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
335+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.613c31543a6423df81
336+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 4a.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
337337
`
338338
),
339339
buildProdNormal(

‎test/cases/declaration-variable.postcss.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ module.exports = test(
139139
`,
140140
compose(
141141
onlyMeta('meta.version.webpack >= 5'),
142-
assertCssAndSourceMapContent('main.bc8832a6d2fb1aa49cc0.css'),
142+
assertCssAndSourceMapContent('main.368e6f956d0703b9fef8.css'),
143143
outdent
144144
)`
145145
/src/index.scss
@@ -154,8 +154,8 @@ module.exports = test(
154154
-:- ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 2:52 ░░░░░░; }⏎
155155
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
156156
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
157-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.bc8832a6d2fb1aa49cc
158-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
157+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.368e6f956d0703b9fef
158+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 8.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
159159
`
160160
),
161161
buildProdNormal(
@@ -321,7 +321,7 @@ module.exports = test(
321321
`,
322322
compose(
323323
onlyMeta('meta.version.webpack >= 5'),
324-
assertCssAndSourceMapContent('main.3f82c05d1ce236b32525.css'),
324+
assertCssAndSourceMapContent('main.8c5f319b6c3e56b9ea7a.css'),
325325
outdent
326326
)`
327327
/src/index.scss
@@ -336,8 +336,8 @@ module.exports = test(
336336
-:- ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 2:58 ░░░░░░░░░░░░; }⏎
337337
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
338338
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
339-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.3f82c05d1ce236b3252
340-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 5.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
339+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.8c5f319b6c3e56b9ea7
340+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ a.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
341341
`
342342
),
343343
buildProdNormal(
@@ -503,7 +503,7 @@ module.exports = test(
503503
`,
504504
compose(
505505
onlyMeta('meta.version.webpack >= 5'),
506-
assertCssAndSourceMapContent('main.58dfc73fd8848c516663.css'),
506+
assertCssAndSourceMapContent('main.421db6b7d794d4599810.css'),
507507
outdent
508508
)`
509509
/src/index.scss
@@ -518,8 +518,8 @@ module.exports = test(
518518
-:- ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 2:68 ░░░░░░░░░░░░░░░░░░░░░░; }⏎
519519
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
520520
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ ⏎
521-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.58dfc73fd8848c51666
522-
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 3.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
521+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ /*# sourceMappingURL=main.421db6b7d794d459981
522+
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0.css.map*/░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
523523
`
524524
),
525525
buildProdNormal(

0 commit comments

Comments
 (0)