-
Notifications
You must be signed in to change notification settings - Fork 98
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
updated saucelabs and lodash versions to fix security vulnerabilities #231
Conversation
Cool! |
@digitalfrost you have publish permissions for this module, right? Also, hi @admc ! |
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.
👍
@Jonahss - no I do not have publish permissions for this module. |
@digitalfrost Oh ok, I'll see what I can do. |
@axemclion or @Jonahss could you please merge this PR and release? |
Thank you for the ping @mar10. I've just joined Jonathan Lipps as an Appium consultant at Cloud Grey so I'll be back in the testing industry! Will be able to be much more involved and responsive. I was having trouble running the tests for this repository. I will merge your changes and publish now, but double check to make sure that everything works for you. |
published as v9.0.1 |
@Jonahss that was fast, thank you very much! Good to hear this cool tool gets some attention again. |
btw. |
well if you can read the changes made in the newer version, maybe we could easily switch to it? |
Updated lodash fixes: https://nodesecurity.io/advisories/577
Updated saucelabs fixes: https://nodesecurity.io/advisories/593
According to npm audit these should be a non breaking changes