Skip to content
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

Replace event-stream with map-stream #261

Merged
merged 1 commit into from
Dec 6, 2018
Merged

Conversation

dbemiller
Copy link
Contributor

@dbemiller dbemiller commented Nov 30, 2018

Fixes #260.

The map function in event-stream delegates to a package called map-stream: https://github.com/dominictarr/event-stream/blob/master/index.js#L22

This replaces the dependency and uses map-stream directly

@dbemiller dbemiller changed the title Upgrade event-stream to ^4.0.0 Replace event-stream with map-stream Nov 30, 2018
@lucasdavila
Copy link

👍 Thank you for the fixup.

@bransbury
Copy link

Can this be merged and released?

@enrique-ramirez
Copy link

Hi. Is there anything else missing for this to be merged?

@silaike
Copy link

silaike commented Dec 6, 2018

@avevlad Is this project still maintained?

achingbrain added a commit to achingbrain/gulp-connect that referenced this pull request Dec 6, 2018
@avevlad avevlad merged commit 8a2bcbb into avevlad:master Dec 6, 2018
@avevlad
Copy link
Owner

avevlad commented Dec 6, 2018

@dbemiller Thanks!

npm publish gulp-connect@5.7.0

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.

6 participants