Skip to content

web: Remove thread-loader from Webpack

Warren Gifford requested to merge mihir/remove-thread-loader into main

Created by: plibither8

Should be merged before https://github.com/sourcegraph/sourcegraph/pull/39381.


The SentryWebpackPlugin does not work with thread-loader, since thread-loader doesn't have access to webpack options, thus it cannot access query params of the loader. See https://github.com/getsentry/sentry-webpack-plugin/issues/272#issuecomment-942381393.

Secondly, thread-loader itself has not shown any substantial performance or build-time improvements.

With thread-loader Google Chrome 2022-07-28 at 15 22 15 000232@2x
Without thread-loader Google Chrome 2022-07-28 at 15 25 03 000234@2x

Test plan

Builds are passing with green checks without thread-loader.

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading