-
Notifications
You must be signed in to change notification settings - Fork 11
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
docs: update README to contain some basic usage examples #20
Conversation
Thanks! That’s the demo plug-in which I haven’t finished yet. I’ll get it up today and modify this PR
…On Sep 19, 2023 at 7:38 AM -0500, Muhammad Azeez ***@***.***>, wrote:
@bhelx I just notice the example plugins don't actually write total and vowels properties in the output json
You can view, comment on, or merge this pull request online at:
#20
Commit Summary
• 5db8537 docs: update README to contain some basic usage examples
File Changes
(1 file)
• M README.md (184)
Patch Links:
• https://github.com/extism/go-sdk/pull/20.patch
• https://github.com/extism/go-sdk/pull/20.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@mhmd-azeez try this link i just updated it with. It's a new version of count-vowels. However, i've decided to go with something different to demo host functions. For the time being, could you write something small / custom for the host functions section? Maybe just a short example doesn't need to be a working demo. I can come back when my other demo plugin is done and we can re-do it. |
Actually I should have a demo for host functions ready by tomorrow if you want to wait. I'm going to write up the example for ruby tomorrow morning and get some feedback. |
@bhelx i think this is ready now |
@mhmd-azeez excellent! we'll make any small tweaks we need in follow up PRs |
@bhelx I just notice the example plugins don't actually write
total
andvowels
properties in the output json