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
fix(README): Update axios example #2464
Conversation
This was added with nock#1711 but [has changed with axios v1](axios/axios#5277)
@@ -1540,6 +1540,9 @@ import test from 'ava' // You can use any test framework. | |||
// | |||
// References: | |||
// https://github.com/axios/axios/pull/5277 | |||
// https://github.com/nock/nock/issues/699#issuecomment-272708264 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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!
@@ -1561,13 +1564,15 @@ test('can fetch test response', async t => { | |||
For Nock + Axios + Jest to work, you'll have to also adapt your jest.config.js, like so: | |||
|
|||
```js | |||
moduleNameMapper: { | |||
const config = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
🎉 This PR is included in version 13.3.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This was added with #1711 but has changed with axios v1