-
-
Notifications
You must be signed in to change notification settings - Fork 228
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
fix: restore IncrementResponse type #397
Conversation
@shenie would you mind trying this out and confirm that it fixes the typing error? I think the commands are going to be something along these lines:
(The Once you're done testing the fix, I believe a regular |
I'd also like to figure out why our integration test didn't catch this... |
done. it worked |
awesome; I'll go ahead and ship this now and look into the tests later |
Ok, 6.11.2 is now live on npm with the fix. Thanks for bring it to our attention and helping me test it! |
Thanks for fixing this so quickly @nfriedly! |
Restores
IncrementResponse
as an alias ofClientRateLimitInfo
fixes express-rate-limit/rate-limit-redis#192 (hopefully)