-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
implemented & added tests for hash operations HSET, HKEYS, HGET, HDEL, HGETALL #125
Conversation
You don't have to open a new PR only to add another commit, simply pushing the test commit to your branch would have updated the old PR. (This is also why it's generally preferred to make a branch on your fork first, and open a pull request from the branch to the original master. 😉) Thanks for the PR by the way. |
Thank you for the advice. |
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.
Hi @popaaaandrei, sorry for missing this PR. I'd be happy to merge this if we can resolve the conflicts.
@tanner0101 done; rebased again |
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.
Thanks!
Hey @popaaaandrei, you just merged a pull request, have a coin! You now have 1 coins. |
There are more hash operations but these are the basic ones.
Also tried to document as much as possible.