Configured the thread-loader to keeping workers alive in development mode

[sadkovoy]

#4346

[facebook-github-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

[facebook-github-bot]

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

[gaearon]

Thanks! This does feel better.

The rebuild performance is still somewhat slower on next branch than on master. Would you like to look into why?

[sadkovoy]

Ok, I'll try to compare and debug. Could you close the related issue?

[viankakrisna]

cmiiw, but isn't this make more sense in app thread-loader instead of dependencies thread-loader?

create-react-app/packages/react-scripts/config/webpack.config.dev.js

Line 225 in 6a663b8

require.resolve('thread-loader'),

do we retranspile dependencies on every rebuild?

[iansu]

@viankakrisna That is an excellent point. Nice catch. I think this should either be used in both places or just for the app code. What do other people think?

[gaearon]

Should be used in both places I imagine.

[sadkovoy]

Oh, thank you! PR: #4582