Skip to content

Document the rest of the C API#1959

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:c-api-docs-more
Jul 2, 2020
Merged

Document the rest of the C API#1959
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:c-api-docs-more

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This commit fills out documentation for all remaining functions in the C
API, and additionally enables "warn if undocumented" which will fail CI
since warnings are also treated as errors.

@github-actions github-actions Bot added the wasmtime:c-api Issues pertaining to the C API. label Jul 1, 2020
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 1, 2020

Subscribe to Label Action

cc @peterhuene

Details This issue or pull request has been labeled: "wasmtime:c-api"

Thus the following users have been cc'd because of the following labels:

  • peterhuene: wasmtime:c-api

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

Copy link
Copy Markdown
Member

@kubkon kubkon left a comment

Choose a reason for hiding this comment

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

🎉🎉🎉

Comment thread crates/c-api/include/doc-wasm.h Outdated
Comment thread crates/c-api/include/doc-wasm.h Outdated
Comment thread crates/c-api/include/wasmtime.h Outdated
Comment thread crates/c-api/include/wasmtime.h Outdated
This commit fills out documentation for all remaining functions in the C
API, and additionally enables "warn if undocumented" which will fail CI
since warnings are also treated as errors.
@alexcrichton alexcrichton merged commit 47a218f into bytecodealliance:main Jul 2, 2020
@alexcrichton alexcrichton deleted the c-api-docs-more branch July 2, 2020 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:c-api Issues pertaining to the C API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants