Skip to content

fix: use typeof instead of instanceof#1850

Merged
paulmelnikow merged 2 commits intonock:masterfrom
salmanm:master
Jan 8, 2020
Merged

fix: use typeof instead of instanceof#1850
paulmelnikow merged 2 commits intonock:masterfrom
salmanm:master

Conversation

@salmanm
Copy link
Copy Markdown
Contributor

@salmanm salmanm commented Jan 7, 2020

@offirgolan
Copy link
Copy Markdown

👋 this is a blocking issue for us as Nock isn't handling stream responses correctly as the common.isStream util always returns false. Would love to get this in and released asap 😅

Copy link
Copy Markdown
Member

@paulmelnikow paulmelnikow left a comment

Choose a reason for hiding this comment

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

Let's link to the place where this was discussed.

Co-Authored-By: Paul Melnikow <github@paulmelnikow.com>
@salmanm
Copy link
Copy Markdown
Contributor Author

salmanm commented Jan 8, 2020

Thanks @paulmelnikow, lets ship it.

@paulmelnikow paulmelnikow merged commit c7363e5 into nock:master Jan 8, 2020
@paulmelnikow
Copy link
Copy Markdown
Member

Thanks for the report and the fix!

@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Jan 8, 2020

🎉 This PR is included in version 11.7.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

nikaspran added a commit to nikaspran/nock that referenced this pull request Feb 10, 2020
juninmd pushed a commit to juninmd/nock that referenced this pull request Mar 21, 2026
Fix a regression in 11.7.1 due to Jest having different globals: jestjs/jest#2549. Check functions using typeof instead of instanceof.
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.

5 participants