-
Notifications
You must be signed in to change notification settings - Fork 568
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
Promise doesn't get rejected #256
Comments
I also think errors should reject the promise! (^^♪ |
Hey @remhume, I opened a PR #297 before I realized you already had one open. If they go with yours maybe you can at least make good use of my unit test : ) |
@stuartpb nice, who do we need to ping to get it merged? |
Thanks everyone for showing demand for rejecting on errors. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey, I just noticed that twit seems to not reject a promise when, for example, the rate limit exceeded in a get request. Instead it just resolves the promise and has the error(s) as an array in the result.data.
I'm not sure if it's intended like that but I expected it to reject the promise when I don't get the data I requested (my promise chain should break when that happens).
The text was updated successfully, but these errors were encountered: