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

Adressing feedback from upstream sync #27

Merged
merged 2 commits into from
Nov 7, 2024

Conversation

danbugs
Copy link
Collaborator

@danbugs danbugs commented Nov 6, 2024

  • clarified comment on topic function.
  • added portability comment onto metadata to avoid it being used as an escape-hatch.
  • removed get-topics func in favour of using wasi-config for runtime configuration.

- clarified comment on `topic` function.
- added portability comment onto `metadata` to avoid it being used as an escape-hatch.
- removed `get-topics` func in favour of using wasi-config for runtime configuration.
@danbugs danbugs added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 6, 2024
Copy link
Collaborator

@thomastaylor312 thomastaylor312 left a comment

Choose a reason for hiding this comment

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

Thanks for pulling in all this feedback!

@danbugs danbugs merged commit 55c13e8 into WebAssembly:main Nov 7, 2024
1 check passed
@danbugs danbugs deleted the danbugs/upstream-meeting-feedback branch November 7, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants