search

mdevilliers / redishappy

Redis Sentinel high availabillity daemon
Apache License 2.0
114 stars 23 forks source link

Compatibility with latest therealbill/libredis #40

Closed scottgeary closed 8 years ago

scottgeary commented 8 years ago

Introduced during the slight code refactor in: https://github.com/therealbill/libredis/commit/9ab2d8db5c00d18d193cbbb396e9d73bc2924866

Some future PRs will require an up-to-date version of this repo.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.8%) to 74.751% when pulling 6a3dd196f43e6ec17e3fea23776b9ae436acb165 on vend:vend_use_latest_libredis into c1ad8e19f0b16a4fba6c3147135b46a7c3fc74a4 on mdevilliers:master.

mdevilliers commented 8 years ago

Thanks,

I had a look at the other branches you have been working on and they look pretty good.

Would you like to be added as a collaborator on this project with push access?

Mark

scottgeary commented 8 years ago

Thanks - that would be awesome :+1:

Still would like your eyes on the various PRs though (starting to push them now), if that's okay?

mdevilliers commented 8 years ago

I've added you as a collabarator - so you should have commit/merge rights.

This one looks awesome - so merging.