-
-
Notifications
You must be signed in to change notification settings - Fork 27k
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
Add WASM support #8303
base: main
Are you sure you want to change the base?
Add WASM support #8303
Conversation
Can you explain what you consider to be "proper" WASM support? |
currently it just doesn't work, you can check it with test attached |
Hey, CRA team, could you really include this? It has been over a year already, there is a ton of useful WASM NPM packages out there and CRA still fails miserably to build it out of the box. |
What is missing here? |
I am in agreement with @usktv. It would be great if create-react-app works out of the box with WASM. I'm running into some painful WASM integration with my an app that was created from the create-react-app. FYI, using a vanilla react app (without create-react-app) is really easy to use WASM and i hope it'll be easy with create-react-app generator |
Hey @ianschmitz @iansu @mrmckeb @petetnt - can you please update the thread with the reason for this not getting through yet? What else needs to be done? It is such a pain to consume wasm from CRA today. |
In fact it is true (@parthopdas) in the end a simple rule on the webpack would be enough or rather a solution in parcel style. |
This PR adds proper WASM support with tests