GoogleChromeLabs / worker-plugin

👩‍🏭 Adds native Web Worker bundling support to Webpack.
https://npm.im/worker-plugin
Apache License 2.0
1.91k stars 79 forks source link

Document "dynamic" output.filename requirement #64

Closed JEphron closed 4 years ago

JEphron commented 4 years ago

A static value for output.filename in the Webpack config will cause issues when multiple workers are used. Webpack does warn about this by printing WARNING in Conflict: Multiple assets emit different content to the same filename, but it took some digging to discover why.

Issue https://github.com/GoogleChromeLabs/worker-plugin/issues/24 discusses this problem, but I would have spent less time being confused if the "dynamic filename" requirement was mentioned in the documentation.

googlebot commented 4 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

googlebot commented 4 years ago

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

developit commented 4 years ago

I'm actually wondering if we should be using output.chunkFilename here instead.