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.

Sign up for GitHub

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

Jump to bottom

Add ruby library to list #16

Closed
wants to merge 2 commits into from
Closed

Add ruby library to list #16

wants to merge 2 commits into from

Conversation

sandstrom
Copy link
Contributor

@sandstrom sandstrom commented May 3, 2023 •
edited
Loading

Update the 'language support' section on this website:
https://wasmtime.dev/

@sandstrom
Copy link
Contributor Author

ping @K0R7 @fitzgen

fitzgen
fitzgen reviewed Jan 24, 2024
View reviewed changes
Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

In wasmtime/README.md we mention the gem rather than the repo. IIUC, that is what users will care more about. Should we switch the wording here? If we want to reference the repo rather than the gem, then we should change wasmtime/README.md to match.

@sandstrom
Copy link
Contributor Author

@fitzgen I've updated to what I think you are asking for.

@sandstrom sandstrom requested a review from fitzgen January 25, 2024 10:30
fitzgen
fitzgen approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

Thanks! And sorry that this took an embarrassingly long time to merge 😬

@fitzgen fitzgen mentioned this pull request Jan 25, 2024
Merged
@fitzgen
Copy link
Member

fitzgen commented Jan 25, 2024

Merging this as part of #23. Thanks again!

@fitzgen fitzgen closed this Jan 25, 2024
@sandstrom sandstrom deleted the patch-1 branch January 26, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fitzgen fitzgen fitzgen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sandstrom @fitzgen