-
Notifications
You must be signed in to change notification settings - Fork 35
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
build(deps): Updated deps to the latest version #347
Conversation
- Drop standard-version as it is now deprecated - Update mocha binaray name in test scripts
Bumping to newer version of |
For the changes in 0eb4b89 I ran: ./node_modules/eslint/bin/eslint.js lib/ test/ --fix Then manually disabled the following rules in the appropriate files
|
Ran integration tests manually:
|
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.
LGTM
Just had a LUG customer asking about npm audit
warnings, so it is a very timely PR
After these changes running npm audit returns:
Still need to investigate possible breaking changes in ssh2 1.x.x in #338