-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Go as an embedding to the book #1481
Add Go as an embedding to the book #1481
Conversation
Also take this time to list out all embeddings in the README of wasmtime itself.
Since the README is so prominent, I'm curious how others feel about the links here. I chose to link to the book docs as well as the native language package (where possible), but it might perhaps be better to link to the API documention uniformly? What do others think? |
I think including the embeddings in the README is a very good idea! And I also like how you linked to things, so 👍 from me! |
Updated! |
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!
README.md
Outdated
* **[.NET]** - the [`Wasmtime` NuGet package] | ||
* **[Go]** - the [wasmtime-go repository] | ||
|
||
[Rust]: https://bytecodealliance.github.io/wasmtime/embed-rust.html |
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.
This should point to lang-rust.html now.
Also take this time to list out all embeddings in the README of wasmtime
itself.