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

fix(examples): Hello world component examples #1856

Merged
merged 6 commits into from
Apr 10, 2024
Merged

Conversation

brooksmtownsend
Copy link
Member

@brooksmtownsend brooksmtownsend commented Apr 9, 2024

Feature or Problem

This PR removes the outdated actor example, updates the wit-bindgen dependency and wit worlds in the Rust http hello world component, and commits the "finalized" version of our quickstart as the http-keyvalue-counter example. I plan to do the exact same thing as our other examples, but I'll keep the PRs as small as I can.

I refactored some of the manifest names and component names to be more consistent and shorter so we can have nice generated IDs with wadm.

I went ahead and just ran a wit-deps update in the Python, TinyGo, and Typescript examples directories but I need to validate that worked properly.

Related Issues

Release Information

I'd love to get both the http hello world and keyvalue counter examples released eventually, to serve as a common base.

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
@brooksmtownsend brooksmtownsend requested a review from a team as a code owner April 9, 2024 20:57
@brooksmtownsend brooksmtownsend changed the title fix(examples): Rust hello world component example fix(examples): Hello world component examples Apr 9, 2024
@brooksmtownsend brooksmtownsend requested a review from a team as a code owner April 10, 2024 14:20
@brooksmtownsend brooksmtownsend enabled auto-merge (rebase) April 10, 2024 14:23
@brooksmtownsend brooksmtownsend force-pushed the fix/1.0-examples branch 3 times, most recently from aec55b7 to dc62d88 Compare April 10, 2024 16:37
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: go example

Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix go

Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix go

Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix(examples): update python component kv dep

Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Copy link
Contributor

@vados-cosmonic vados-cosmonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@brooksmtownsend brooksmtownsend merged commit 9fffe45 into main Apr 10, 2024
61 checks passed
@brooksmtownsend brooksmtownsend deleted the fix/1.0-examples branch April 10, 2024 17:52
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.

None yet

4 participants