Skip to content

Commit 48f6805

Browse files
silverwinddelvh
andauthored
Remove worker-loader (#23548)
[`worker-loader`](https://github.com/webpack-contrib/worker-loader) is deprecated since webpack 5 which can load workers without it now, so remove it. I think it was already dysfunctional because the regex does not match our current worker scripts: ``` web_src/js/features/eventsource.sharedworker.js web_src/js/features/serviceworker.js web_src/js/serviceworker.js ``` I did confirm that eventsource worker still loads via simple `console.log` inside the script. Co-authored-by: delvh <[email protected]>
1 parent fe8b6cb commit 48f6805

File tree

3 files changed

+0
-121
lines changed

3 files changed

+0
-121
lines changed

package-lock.json

-108
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747
"webpack-cli": "5.0.1",
4848
"workbox-routing": "6.5.4",
4949
"workbox-strategies": "6.5.4",
50-
"worker-loader": "3.0.8",
5150
"wrap-ansi": "8.1.0"
5251
},
5352
"devDependencies": {

webpack.config.js

-12
Original file line numberDiff line numberDiff line change
@@ -111,18 +111,6 @@ export default {
111111
exclude: /node_modules/,
112112
loader: 'vue-loader',
113113
},
114-
{
115-
test: /\.worker\.js$/,
116-
exclude: /monaco/,
117-
use: [
118-
{
119-
loader: 'worker-loader',
120-
options: {
121-
inline: 'no-fallback',
122-
},
123-
},
124-
],
125-
},
126114
{
127115
test: /\.js$/,
128116
exclude: /node_modules/,

0 commit comments

Comments
 (0)