Skip to content

Conversation

@john-says-hi
Copy link

Changed couple things to compile for wasm32.

@asaphaaning
Copy link
Owner

Hi @johnwaltonvi, thanks for contributing!

Would you mind splitting up the work so that this PR deals only with WASM support?

@john-says-hi
Copy link
Author

Hi! Yes i can try to do this tomorrow. First time PR. Thanks for making the lib.

@asaphaaning
Copy link
Owner

asaphaaning commented Jun 14, 2025

Hi! Yes i can try to do this tomorrow. First time PR. Thanks for making the lib.

Great! Generally I think you'll find it helpful to remember that your PR should do one thing only and do that thing well. If you have more things to contribute, open another PR!

You're welcome, glad you find it useful.

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