-
Notifications
You must be signed in to change notification settings - Fork 127
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
chore(deps): update ioredis to v5.0.4 #1172
Conversation
This would be great to get merged and published 🙏 |
Hi @vamsiampolu and @crobinson42 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey I merged #1166 first as it didn't require any changes to the README to land.
Everything else in the PR looks good 👍
Is there something I can do to help with |
No, that task isn't supposed to run on forks and I've tried to make it only run on upstream multiple times but it's stubborn D: Thanks for the bonus fixes you pushed to the e2e suite btw ❤️ I'll merge this and fix the build errors on main! |
🎉 This PR is included in version 8.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
peerDependencies
to5.x
anddevDependencies
to5.0.4
.ioredis/built/Command
instead ofioredis/built/command
to fix tests.PromiseContainer
and any references to it.