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

Update dependencies to fix `npm audit` warnings #8

Merged
merged 2 commits into from Oct 29, 2018

Conversation

Projects
None yet
2 participants
@dhleong
Contributor

dhleong commented Oct 18, 2018

Also uses the ^version syntax for dependencies so clients can
potentially resolve future such audit warnings by updating dependencies
themselves.

I did not update gulp because I'm not super familiar with it, and it was
a major version change, so there could be breaking API changes.

Thanks for this project, by the way!

Update dependencies to fix `npm audit` warnings
Also uses the `^version` syntax for dependencies so clients can
potentially resolve future such audit warnings by updating dependencies
themselves.

I did not update gulp because I'm not super familiar with it, and it was
a major version change, so there could be breaking API changes.
@Feiyang1

This comment has been minimized.

Owner

Feiyang1 commented Oct 29, 2018

Thank you for taking interest and effort in this project! I checked the gulp@4 changes, this project should continue to work without any change. If you could update the gulp version as well, It will be good to go.

@dhleong

This comment has been minimized.

Contributor

dhleong commented Oct 29, 2018

Done!

@Feiyang1 Feiyang1 merged commit a5561c4 into Feiyang1:master Oct 29, 2018

@Feiyang1

This comment has been minimized.

Owner

Feiyang1 commented Oct 29, 2018

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment