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

Drop dependency on deprecated gulp-util #77

Merged
merged 1 commit into from
Aug 30, 2018
Merged

Drop dependency on deprecated gulp-util #77

merged 1 commit into from
Aug 30, 2018

Conversation

TheDancingCode
Copy link
Contributor

gulp-util has been recently deprecated. Continuing to use this dependency may prevent the use of your library with the latest release of Gulp 4 so it is important to replace gulp-util.

See:

@coveralls
Copy link

coveralls commented Dec 30, 2017

Coverage Status

Coverage remained the same at 95.652% when pulling 23416c3 on TheDancingCode:pr/replace-gutil into d1d6d30 on ben-eb:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.652% when pulling 23416c3 on TheDancingCode:pr/replace-gutil into d1d6d30 on ben-eb:master.

Copy link

@demurgos demurgos left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ben-eb
Could you merge and publish it?

@TheDancingCode
Copy link
Contributor Author

@ben-eb
Did you get a chance to look at this?

@TheDancingCode
Copy link
Contributor Author

Another friendly reminder.

@TheDancingCode
Copy link
Contributor Author

If you need help maintaning this package, I'd be glad to help.

@rejas
Copy link
Collaborator

rejas commented May 2, 2018

A third friendly reminder @ben-eb :-) Also would offer to help maintain this package if you dont have the time.

@rejas
Copy link
Collaborator

rejas commented May 31, 2018

Hi @ben-eb still hoping to get a reaction from you regarding this PR and the future of this package.

@fictionic
Copy link

He's definitely active on GitHub. I guess he doesn't check his notifications

@kykint
Copy link

kykint commented Aug 15, 2018

What about merging this PR?

@rejas
Copy link
Collaborator

rejas commented Aug 15, 2018

Maybe pinging @ben-eb helps....

Copy link
Collaborator

@rejas rejas left a comment

Choose a reason for hiding this comment

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

lgtm!

@rejas rejas merged commit 2472b9d into ben-eb:master Aug 30, 2018
@demurgos
Copy link

Great, let me know when it is published to npm. 15 packages to go to complete the goals of the migration issue 🎉

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