-
Notifications
You must be signed in to change notification settings - Fork 61
removed deprecated gulp-util #50
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
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.
nice!
@zbennett10 3.9.1 has been published to npm. Thank you for being on top of these things! |
@scniro you're welcome! Thank you for the tool! |
@@ -30,7 +30,8 @@ | |||
], | |||
"dependencies": { | |||
"clean-css": "4.1.9", | |||
"gulp-util": "3.0.8", | |||
"fancy-log": "^1.3.2", |
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.
This should probably be a devDep (only used in gulpfile.js)
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.
Indeed! Good catch.
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.
We want to be careful about where dependencies go so plugin install size can be reduced as much as possible.
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.
@phated Nice catch, now that plugin-error
lives on it's own I can definitely fix as you suggested. Will be incoming soon...
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.
fixed with 3.9.2
This pr replaces the recently deprecated gulp-util package with corresponding dependencies.
For more info see: https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
Thanks!