-
Notifications
You must be signed in to change notification settings - Fork 8
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
ioredis v5 is not supported yet #20
Comments
I'm running this library with ioredis v5 and it seems to work properly. |
It does not work with v5 with typescript. Here is one not so great, but working alternative for TypeScript to force In ...
"dependencies": {
"@microfleet/ioredis-lock": "^5.1.0",
"ioredis": "^5.3.2",
....
}
"overrides": {
"@microfleet/ioredis-lock": {
"ioredis": "$ioredis"
}
}, |
any updates here? discovered i had a repo running v5 redis with v4 redis lock while swapping it over from yarn to npm today. once i used npm i realized i have conflicting peer dependencies and can't install. would love to just bump to 5 instead of downgrading redis itself but for now im just downgrading to be safe. |
Main problem is that ioredis 5 uses native promises and 4 uses provider (used bluebird at the time). pr is welcome with support for v5 / removal of bluebird - I can release it if you have time to work on it |
@AVVS awesome, helpful to know the issue. that makes perfect sense. will see if i have some time. a bit swamped at work atm but would love to help out if i find time. |
ioredis
v5 is not supported yet withioredis-lock
. What is a plan around it?The text was updated successfully, but these errors were encountered: