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 gulp-util with alternatives #124

Merged
merged 5 commits into from Jan 1, 2018

Conversation

Projects
None yet
4 participants
@ohbarye
Contributor

ohbarye commented Dec 30, 2017

Hello gulp-notify maintainers.

Following gulpjs/gulp-util#143, I dropped dependency on gulp-util.

closes #123

Thanks!

Show outdated Hide outdated package.json Outdated
@demurgos

This comment has been minimized.

Show comment
Hide comment
@demurgos

demurgos Dec 30, 2017

Thanks for the PR, I had some issues with the tests and did not have time to look into this yesterday.

@mikaelbr Could we have this merged?

demurgos commented Dec 30, 2017

Thanks for the PR, I had some issues with the tests and did not have time to look into this yesterday.

@mikaelbr Could we have this merged?

@demurgos demurgos referenced this pull request Dec 30, 2017

Open

Ecosystem migration #143

94 of 106 tasks complete

@mikaelbr mikaelbr merged commit fc51ff2 into mikaelbr:master Jan 1, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mikaelbr

This comment has been minimized.

Show comment
Hide comment
@mikaelbr

mikaelbr Jan 1, 2018

Owner

🎉 thanks!

Owner

mikaelbr commented Jan 1, 2018

🎉 thanks!

@ohbarye ohbarye deleted the ohbarye:replace-gulp-util branch Jan 1, 2018

@ohbarye

This comment has been minimized.

Show comment
Hide comment
@ohbarye

ohbarye Jan 1, 2018

Contributor

@mikaelbr Thank you too!

Contributor

ohbarye commented Jan 1, 2018

@mikaelbr Thank you too!

@phil-r

This comment has been minimized.

Show comment
Hide comment
@phil-r

phil-r Jan 2, 2018

@mikaelbr Can this be published to npm? ^_^ thanks!

phil-r commented Jan 2, 2018

@mikaelbr Can this be published to npm? ^_^ thanks!

@mikaelbr

This comment has been minimized.

Show comment
Hide comment
@mikaelbr

mikaelbr Jan 4, 2018

Owner

Indeed!

▶ npm publish
+ gulp-notify@3.1.0
Owner

mikaelbr commented Jan 4, 2018

Indeed!

▶ npm publish
+ gulp-notify@3.1.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment