-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(wasm): C stub #118
feat(wasm): C stub #118
Conversation
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.
I'm wondering if C has some modern linters/formaters?
Currently, we don't have any checks in place. For now, the code just needs to be simple and work. If we decide to add those checks later on, we can update and improve the code at that time. |
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.
Looks good to me 🥳
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.
LGTM
Description
Implement all events defined in Hermes
.wit
file exports in C and prepare a builder to build the wasm module.Related Issue(s)
Closes #117
Description of Changes
.wasm
componentPlease confirm the following checks