-
Notifications
You must be signed in to change notification settings - Fork 76
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
Remove dependency on gulp #153
Comments
@eseliger gulp should be a peer dependency as the package requires it, however, as a peer dependency, we can update the module with our own security updates as needed. |
If I get your point correctly, you are worried about using the wrong version of gulp with this plugin? If not, please correct me. |
As per gulp plugin guidelines, there should be no dependency on gulp itself, for a good reason. So removing it as a dependency and adding it as a devDependency should be viable in my opinion. I can open a PR with this if you agree
The text was updated successfully, but these errors were encountered: