search

vapor / redis

Vapor provider for RediStack
MIT License
459 stars 58 forks source link

implemented & added tests for hash operations HSET, HKEYS, HGET, HDEL, HGETALL #125

Closed popaaaandrei closed 5 years ago

popaaaandrei commented 5 years ago

There are more hash operations but these are the basic ones. Also tried to document as much as possible.

vzsg commented 5 years ago

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.

popaaaandrei commented 5 years ago

Thank you for the advice.

popaaaandrei commented 5 years ago

@tanner0101 done; rebased again

penny-coin commented 5 years ago

Hey @popaaaandrei, you just merged a pull request, have a coin!

You now have 1 coins.