-
Notifications
You must be signed in to change notification settings - Fork 59
Blog post describing updates to Valkey Modules Rust SDK #250
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
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.
There is lots of good stuff here.
It feels a little like release notes in the current state, especially towards the end.
What would make it a good blog post is a few paragraphs that tie together why these changes are important (is it just keeping up with changes in Valkey? does it fill in the SKD gaps?, etc.).
At the end it goes into bullet point mode rather than narrative too. Maybe make the ideas for further development into a paragraph and the list of contributions into a sentence.
It needs a conclusion paragraph as well.
I also left some specific wording and grammar notes.
agree on the |
8254955
to
80f5e10
Compare
Signed-off-by: Dmitry Polyakovsky <[email protected]>
…alkey-io#249) The documentation tiles on the homepage were stacking awkwardly at screen widths between 768px and 1090px. Modified the grid layout to create a more natural 2x2 layout at these intermediate screen sizes, ensuring consistent spacing and better visual organization. <img width="1023" alt="Screenshot 2025-04-24 at 4 53 59 PM" src="https://github.com/user-attachments/assets/237bce45-1cec-46ad-925a-766d0627fd93" /> <img width="480" alt="Screenshot 2025-04-24 at 4 54 13 PM" src="https://github.com/user-attachments/assets/ee8e564a-44d1-45f4-914b-248787ae9340" /> Signed-off-by: Daniel Phillips <[email protected]> Signed-off-by: Dmitry Polyakovsky <[email protected]>
Signed-off-by: dmitrypol <[email protected]> Signed-off-by: Dmitry Polyakovsky <[email protected]>
Signed-off-by: Dmitry Polyakovsky <[email protected]>
978870c
to
5848429
Compare
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.
Looks very nice. A few small issues but I'm comfortable publishing on the updated pub date in the TOML.
### Description Blog post describing updates to Valkey Modules Rust SDK ### Issues Resolved <!-- List any issues this PR will resolve. --> <!-- Example: closes #1234 --> ### Check List - [ ] Commits are signed per the DCO using `--signoff` By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License. --------- Signed-off-by: Dmitry Polyakovsky <[email protected]> Signed-off-by: Daniel Phillips <[email protected]> Signed-off-by: dmitrypol <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Co-authored-by: Daniel Phillips <[email protected]> Co-authored-by: Dmitry Polyakovsky <[email protected]> Signed-off-by: Shai Zarka <[email protected]>
Description
Blog post describing updates to Valkey Modules Rust SDK
Issues Resolved
Check List
--signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.