Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support web workers #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EtaiG
Copy link

@EtaiG EtaiG commented Dec 28, 2020

Sometimes, people want to use the code in web-workers.

Specifically, one of my relatives wanted to use this library in a Wix Corvid application. They need to use a target compiled for web-workers in order for it to work.
This PR creates a worker target.

After the build, it would probably be good to add a link to unpkg with the compiled worker target, to make it easier

@BehindTheMath
Copy link
Owner

BehindTheMath commented Dec 28, 2020

@EtaiG Instead of creating a new Webpack config, would something like this work?

@EtaiG
Copy link
Author

EtaiG commented Dec 29, 2020

@EtaiG Instead of creating a new Webpack config, would something like this work?

Theoretically yes, and it would be a better solution.
I haven't tried any of the workarounds yet, but webpack/webpack#6525 (comment) mentions that they are not enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants