search

dabroek / node-cache-manager-ioredis

Redis store for node-cache-manager using IORedis.
MIT License
59 stars 29 forks source link

Supports ttl constructor option with external redis instances #15

Open wbk opened 3 years ago

wbk commented 3 years ago

This addresses the issue in #14

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.1%) to 93.333% when pulling 85eb75de1700f2d22a448629601d0c2c86e064d7 on wbk:master into 3bf4514e2e0c0b76126bb1997103b7555ff4b003 on dabroek:master.

coveralls commented 3 years ago

Coverage Status

Coverage decreased (-0.7%) to 92.5% when pulling 5cbf5933aa265a345c63897fd7c3c62119403b7d on wbk:master into 3bf4514e2e0c0b76126bb1997103b7555ff4b003 on dabroek:master.

dylanseago commented 3 years ago

Can we get this merged? I can confirm this PR fixes the issue

omerautodesk commented 2 years ago

Is this going to be merged, or can we declare this package as deprecated? @dabroek @marcoreni

luluhoc commented 1 year ago

@dabroek

jeanbaptistesouvestre commented 1 year ago

Hello, would it be possible to merge this ? Thanks ! 🙏

surahj commented 1 year ago

Is it still possible to merge?

artyomtrubchik commented 1 year ago

guys, could you review this pr and merge 🙏