-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to ioredis 5 #32
Comments
Can you describe what issues you have encountered using this package out of the box and/or why specifically you are looking for support for ioredis 5.* ? From what I understand this just uses get/set, so I'm curious as to your situation. |
In our project team we have the policy to update libraries (and nodejs) to latest version to ensure to be always up to date, have all corrections, and vulnerabilities fixes. More, in a project that depends of ioredis and "node-cache-manager-ioredis" having two version of ioredis increase the bundle by having two version of a library server side (ie: when using lambda). ioredis 4.* is already 6 months old. We don't encounter issue at today. |
Hi @phoenix741 Did a rewrite of this package with |
personally, I'd just remove the direct dependency on It makes dependency-management far simpler, and at worse, only adds about 2 extra lines of code. |
Hi,
Can you upgrade to ioredis 5.* ?
Thanks
The text was updated successfully, but these errors were encountered: