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
revert: change type of AxiosResponse to any #4186
Conversation
Yeah you’re right I forgot to mention that one, just mentioned the initial PR that brought this issue up. Thanks for the heads up! |
My bad xD
@ImRodry please can you have a look at the failing tests? |
@jasonsaayman apologies for the failing tests, they should be good now! |
I wasn't able to test this locally because for some reason it relies on Firefox being installed which it's not on my computer so please approve the workflows again to see if more errors arise |
Hey @jasonsaayman, thanks for merging the PR! Can we get an ETA on when v0.24 will be released? |
Will try and get it out tonight |
* revert: change type of AxiosResponse to any * types: add back default types My bad xD * Remove redundant default params
This pull request fixes #4141 by reverting the change implemented in #3002 (changed the type of AxiosResponse to any).