-
Notifications
You must be signed in to change notification settings - Fork 390
fix(deps): remove stream utils #7120
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
Conversation
I had rebased with the wrong npm version
caed637
to
f1a86db
Compare
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.
LGTM. I rebased this and fixed up a few weeks of conflicts, and tested the ntl deploy
flows locally to be extra safe. Thank you!
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.
Let's revert the is-stream
changes, otherwise looks good to me.
@talentlessguy Thanks for getting this PR started, and thanks @serhalp for finishing it up!
Co-authored-by: Nathan Houle <[email protected]>
This reverts commit d812303.
This reverts commit 5d6dd8f.
🎉 Thanks for submitting a pull request! 🎉
Summary
See e18e/ecosystem-issues#156
Removes is-stream, from2-array, through2-filter, flush-write-stream and through2-map in favor of built-in stream utils
For us to review and ship your PR efficiently, please perform the following steps:
passes our tests.
A picture of a cute animal (not mandatory, but encouraged)