dial-once/node-cache-manager-redis

Provide access to raw redis client for advanced usage

cludden opened this issue · 7 comments

cludden commented

would like to be able to access the underlying redis client to use some native functionality, but there is currently no way to access it

jkernech commented

Thanks for your contribution @cludden ! I'll check this out as soon as possible

cludden commented

Thanks @jkernech for the fast response! Any chance you could publish v0.2.0? We're hoping to use some of this functionality in production at my company in the very near future.

jkernech commented

@cludden, I will publish a new version by the end of the week.

I'd like to cover 100% of the code before releasing the v0.2.0 (~94% atm)

https://github.com/dial-once/node-cache-manager-redis/tree/feature/tests

jkernech commented

I didn't manage to get the code coverage to 100% (~95%) but as promised I have published the v0.2.0.

cludden commented

Thanks @jkernech! Let me know if you'd like help on completing coverage, would be more than happy to help out on such a noble goal!

jkernech commented

Thanks @cludden ! I need some help to reach this noble goal so you're welcome to help out :)

We need to find a way to test the connect callback when the redis client throws an error, I will create a dedicated issue but you can find more infos here : #14

jkernech commented

Here you go ; #16