Skip to content

fix(README): Update axios example#2464

Merged
mastermatt merged 4 commits intonock:mainfrom
depoulo:patch-1
Apr 26, 2023
Merged

fix(README): Update axios example#2464
mastermatt merged 4 commits intonock:mainfrom
depoulo:patch-1

Conversation

@depoulo
Copy link
Contributor

@depoulo depoulo commented Apr 17, 2023

This was added with #1711 but has changed with axios v1

@mastermatt mastermatt merged commit 3feef40 into nock:main Apr 26, 2023
//
// References:
// https://github.com/axios/axios/pull/5277
// https://github.com/nock/nock/issues/699#issuecomment-272708264
Copy link
Contributor Author

@depoulo depoulo Apr 27, 2023

Choose a reason for hiding this comment

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

Well this no longer works, which is the whole point of my PR, same for the link right below.

Copy link
Member

Choose a reason for hiding this comment

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

Sorry @depoulo, that was me messing up the rebase. I'll remove those two lines in another PR I have open for approval atm.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no worries, and thanks for merging!


```js
moduleNameMapper: {
const config = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍🏽

@depoulo depoulo deleted the patch-1 branch April 27, 2023 14:38
@github-actions
Copy link

🎉 This PR is included in version 13.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants