-
Notifications
You must be signed in to change notification settings - Fork 137
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
Closing #131. Documentation updates. #151
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of requested changes and questions. Aside from that, let me tell you, the documentation on this looks INCREDIBLE and I think these things are going to pay massive dividends when people are looking up wasmcloud.
Thanks for the tip on using the bullet lists for the arguments. I like how it looks much better than just using a paragraph. |
Looks like a merge may have gone wrong 😬 |
Yep. Not sure what happened, other than me being foolish enough to try and fix the merge using the github editor rather than doing it old school. |
@brooksmtownsend merge problem is all fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG TM
* Redis Expiration addition Signed-off-by: Esteban Gimbernat <egimbernat@sigaba.io> * Cargo fmt fix Signed-off-by: Esteban Gimbernat <egimbernat@sigaba.io> * RedisKV patch version bump Signed-off-by: Esteban Gimbernat <egimbernat@sigaba.io> * Cargo FMT Signed-off-by: Esteban Gimbernat <egimbernat@sigaba.io> Co-authored-by: Esteban Gimbernat <egimbernat@sigaba.io>
fix typos in examples
CC @stevelr