Skip to content

revert: change type of AxiosResponse to any#4186

Merged
jasonsaayman merged 3 commits intoaxios:masterfrom
ImRodry:revert-types-any
Oct 22, 2021
Merged

revert: change type of AxiosResponse to any#4186
jasonsaayman merged 3 commits intoaxios:masterfrom
ImRodry:revert-types-any

Conversation

@ImRodry
Copy link
Copy Markdown
Contributor

@ImRodry ImRodry commented Oct 13, 2021

This pull request fixes #4141 by reverting the change implemented in #3002 (changed the type of AxiosResponse to any).

@ITenthusiasm
Copy link
Copy Markdown

@ImRodry Just for clarity: Technically speaking, you're reverting the changes that appeared after transitioning to #3002 and then #4142. It's a "reversion" of #4142, but that change didn't occur in "isolation", so both PRs are kinda related.

Thanks for opening this.

@ImRodry
Copy link
Copy Markdown
Contributor Author

ImRodry commented Oct 13, 2021

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!

@jasonsaayman
Copy link
Copy Markdown
Member

@ImRodry please can you have a look at the failing tests?

@ImRodry
Copy link
Copy Markdown
Contributor Author

ImRodry commented Oct 21, 2021

@jasonsaayman apologies for the failing tests, they should be good now!

@ImRodry
Copy link
Copy Markdown
Contributor Author

ImRodry commented Oct 21, 2021

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

@jasonsaayman jasonsaayman merged commit 2c9cc76 into axios:master Oct 22, 2021
@ImRodry ImRodry deleted the revert-types-any branch October 22, 2021 09:04
@ImRodry
Copy link
Copy Markdown
Contributor Author

ImRodry commented Oct 24, 2021

Hey @jasonsaayman, thanks for merging the PR! Can we get an ETA on when v0.24 will be released?

@jasonsaayman
Copy link
Copy Markdown
Member

Will try and get it out tonight

mbargiel pushed a commit to mbargiel/axios that referenced this pull request Jan 27, 2022
* revert: change type of AxiosResponse to any

* types: add back default types

My bad xD

* Remove redundant default params
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revert #3002 - set type of data to any

5 participants