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.

Sign up for GitHub

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

Jump to bottom

fix(npm): update Cypress to fix security warnings, fixes #471 #472

Merged

ghiscoding merged 2 commits into master from bugfix/package-security-warnings

Mar 17, 2020

Collaborator

ghiscoding commented Mar 17, 2020

fixes #471

After running a local npm install, I get the following, so I assume it's all good

found 0 vulnerabilities


          fix(npm): update Cypress to fix security warnings

f4abc93

Collaborator Author

ghiscoding commented Mar 17, 2020

hmmm I see 1 of the test fail with this new version.. let me look into that before merging


          fix(tests): make sure variable is number before comparing greater than

2c45a96

ghiscoding merged commit 3a7ef63 into master

ghiscoding deleted the bugfix/package-security-warnings branch

May 18, 2020 00:17

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

Labels

None yet