-
Notifications
You must be signed in to change notification settings - Fork 67
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
chore(examples): add examples #507
chore(examples): add examples #507
Conversation
I'm going to assume that the failing test is unrelated, considering none of the actually important code changed. Possibly a problem with ubuntu latest? |
Yes, just a flake. |
96b3ca3
to
5067303
Compare
Hey @guybedford this is finally ready for review -- example CI is working properly [EDIT] I'm also fine not committing the per-project WASM binaries -- so I could take those out... Would lighten the repo a bit |
00c6834
to
46792e4
Compare
This commit adds examples to the jco, some of which are/will be referenced from the component-model book (https://component-model.bytecodealliance.org/). Signed-off-by: Victor Adossi <[email protected]>
46792e4
to
a54e0a8
Compare
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.
Sorry for the delay here, this is looking great.
I'll aim to merge tomorrow. Just wondering about function inlining but we can skip that too.
examples/components/string-reverse-upper/string-reverse-upper.mjs
Outdated
Show resolved
Hide resolved
Co-authored-by: Guy Bedford <[email protected]>
Signed-off-by: Victor Adossi <[email protected]>
Signed-off-by: Victor Adossi <[email protected]>
416f367
to
8f776c3
Compare
Sorry for the delay, thanks for the detailed review -- I was able to hit everything! |
This commit adds examples to the jco, some of which are/will be referenced from the component-model book (https://component-model.bytecodealliance.org/).