Skip to content

refactor(helpers): optimize the logic of isAxiosError#3546

Merged
jasonsaayman merged 7 commits intoaxios:masterfrom
BlackHole1:refactor-is-axios-error
Dec 22, 2021
Merged

refactor(helpers): optimize the logic of isAxiosError#3546
jasonsaayman merged 7 commits intoaxios:masterfrom
BlackHole1:refactor-is-axios-error

Conversation

@BlackHole1
Copy link
Copy Markdown
Contributor

@BlackHole1 BlackHole1 commented Jan 14, 2021

add the judgment of null

Copy link
Copy Markdown
Contributor

@JasonHK JasonHK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 😄 I should use the isObject function in the first place.

@johachi
Copy link
Copy Markdown
Contributor

johachi commented Apr 14, 2021

Nice! I just discovered this while writing tests for my codebase! Hope this will be merged soon!

@stephenwade
Copy link
Copy Markdown

@jasonsaayman Could you take a look at this please? I just ran into an issue that this would fix (passing null to isAxiosError throws an exception) in my codebase.

@johachi
Copy link
Copy Markdown
Contributor

johachi commented Nov 4, 2021

The forgotten PR 😭

@jasonsaayman
Copy link
Copy Markdown
Member

No longer forgotten :)

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.

5 participants