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

updating dependencies #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

updating dependencies #77

wants to merge 2 commits into from

Conversation

ridhishjain-zepto
Copy link

@ridhishjain-zepto ridhishjain-zepto commented Apr 6, 2020

updating vulnerable and critical dependencies
updated due to #76

@negative0
Copy link
Member

How to check if everything is in order?

@ridhishjain-zepto
Copy link
Author

ridhishjain-zepto commented Apr 10, 2020

The changes worked fine in my machine but they may cause problems if node versions are different. The version I am using is 12.16.1. It might cause errors with earlier versions.

I think Travis CI might be the solution. We can keep this PR on hold and update testing in Travis for change in dependencies
Thoughts?

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.

3 participants