Skip to content
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

Merged

Conversation

vados-cosmonic
Copy link
Contributor

This commit adds examples to the jco, some of which are/will be referenced from the component-model book (https://component-model.bytecodealliance.org/).

@vados-cosmonic
Copy link
Contributor Author

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?

@guybedford
Copy link
Collaborator

Yes, just a flake.

@vados-cosmonic vados-cosmonic force-pushed the chore(examples)=add-examples branch 10 times, most recently from 96b3ca3 to 5067303 Compare October 2, 2024 17:09
@vados-cosmonic
Copy link
Contributor Author

vados-cosmonic commented Oct 2, 2024

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

@vados-cosmonic vados-cosmonic force-pushed the chore(examples)=add-examples branch from 00c6834 to 46792e4 Compare October 2, 2024 17:19
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]>
@vados-cosmonic vados-cosmonic force-pushed the chore(examples)=add-examples branch from 46792e4 to a54e0a8 Compare October 2, 2024 17:22
Copy link
Collaborator

@guybedford guybedford left a 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/add/package.json Outdated Show resolved Hide resolved
examples/components/string-reverse-upper/package.json Outdated Show resolved Hide resolved
examples/components/string-reverse-upper/package.json Outdated Show resolved Hide resolved
examples/components/string-reverse/package.json Outdated Show resolved Hide resolved
examples/components/string-reverse/string-reverse.mjs Outdated Show resolved Hide resolved
@vados-cosmonic vados-cosmonic force-pushed the chore(examples)=add-examples branch from 416f367 to 8f776c3 Compare October 16, 2024 08:20
@vados-cosmonic
Copy link
Contributor Author

Sorry for the delay, thanks for the detailed review -- I was able to hit everything!

@guybedford guybedford merged commit 79f7995 into bytecodealliance:main Oct 18, 2024
18 of 19 checks passed
@vados-cosmonic vados-cosmonic deleted the chore(examples)=add-examples branch October 19, 2024 05:49
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