-
Notifications
You must be signed in to change notification settings - Fork 26
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
#55
Conversation
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.
Looks good.
@tracker1 Could we have this merged and published?
Thanks! |
@TheDancingCode Would you like to be added to the contributors in the package.json? @demurgos If you'd like, I could add you as a contributor so you can push to the project. |
I'll be upfront that being able to perform these sort of maintenance updates when there are some ecosystem-wide change would be good but I will most likely won't have time to deal with issues specific to this library (fixing CI, updating deps is fine, but not sure about issues regarding the features of the lib). |
demurgos, you've been added... after a merge...
travis should do the publish to npm... though it was a bit wonky yesterday. |
I'm frankly not even using gulp anymore, mostly webpack and/or npm scripts for a few years, so I get what you're saying. I don't think there's really a need to add new features, most changes are because of API changes. |
Ok, thanks for the invite. |
Closes #54