socketry / async-redis

MIT License
83 stars 18 forks source link

Implement hashes #17

Closed mhenrixon closed 5 years ago

mhenrixon commented 5 years ago

I hope the spec coverage is sufficient?

ioquatix commented 5 years ago

It looks good.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 105


Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/async/redis/methods/hashes.rb 26 30 86.67%
<!-- Total: 28 32 87.5% -->
Totals Coverage Status
Change from base Build 100: 0.07%
Covered Lines: 369
Relevant Lines: 426

💛 - Coveralls
ioquatix commented 5 years ago

@huba how does this sit with the auto generated API stuff that you were working on?