Skip to content

Add Go as an embedding to the book#1481

Merged
alexcrichton merged 3 commits into
bytecodealliance:masterfrom
alexcrichton:go-bindings
Apr 8, 2020
Merged

Add Go as an embedding to the book#1481
alexcrichton merged 3 commits into
bytecodealliance:masterfrom
alexcrichton:go-bindings

Conversation

@alexcrichton
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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!

Comment thread docs/lang-go.md Outdated
Comment thread README.md Outdated
Comment thread docs/examples-c-wasi.md
@alexcrichton
Copy link
Copy Markdown
Member Author

Updated!

Copy link
Copy Markdown
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!

Comment thread README.md Outdated
* **[.NET]** - the [`Wasmtime` NuGet package]
* **[Go]** - the [wasmtime-go repository]

[Rust]: https://bytecodealliance.github.io/wasmtime/embed-rust.html
Copy link
Copy Markdown
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