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

Add Go as an embedding to the book #1481

Merged
merged 3 commits into from
Apr 8, 2020

Conversation

alexcrichton
Copy link
Member

Also take this time to list out all embeddings in the README of wasmtime
itself.

Also take this time to list out all embeddings in the README of wasmtime
itself.
@github-actions github-actions bot added the wasmtime:docs Issues related to Wasmtime's documentation label Apr 7, 2020
@alexcrichton
Copy link
Member Author

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?

@tschneidereit
Copy link
Member

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!

docs/lang-go.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/examples-c-wasi.md Show resolved Hide resolved
@alexcrichton
Copy link
Member Author

Updated!

Copy link
Member

@sunfishcode sunfishcode left a 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
Copy link
Member

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.

@alexcrichton alexcrichton merged commit 44e897d into bytecodealliance:master Apr 8, 2020
@alexcrichton alexcrichton deleted the go-bindings branch April 8, 2020 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:docs Issues related to Wasmtime's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants