-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove top level await #11
Conversation
@roninjin10 Does this address your concerns from #10? I've re-written the exports along the lines of what we did for |
dd60bb8
to
3adcb71
Compare
This is perfect! Thanks for this. Seeing the related ethereumjs pr and the breaking change here doesn't look too bad |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, great work! 🤩
Will merge and prepare a release!
Hmm, something strange going on here when I try to merge. 🤔 Also, advice from https://stackoverflow.com/questions/53712930/how-to-fix-this-in-git-this-branch-is-out-of-date-with-the-base-branch is not working. Will eventually try to re-submit the PR. |
Replaced by #12 |
This fixes #10 and replaces the current setup with an async function that initializes the WASM and returns the three EC functions as properties of an object.