-
Notifications
You must be signed in to change notification settings - Fork 635
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 yargs to ^17.2.0 #255
base: master
Are you sure you want to change the base?
Conversation
Any update on getting this merged? |
Any updates ? |
With the recent vulnerability on ansi-regex, it is critical that we merge this, since yargs has a dependency on strip-ansi which in turn has a dependency on ansi-regex. |
Hello, the last version of yargs is 17.2.1. Maybe it's work with it ? |
Please merge this! |
Updated to 17.2.1 and appears to be working based on tooling |
Please do merge this and release this - it's showing up in npm audit |
Yes, please merge this! |
Pretty please? |
Can we merge this please? Thanks |
Could we merge this to resolve the vulnerability? https://security.snyk.io/vuln/SNYK-JS-ANSIREGEX-1583908 |
Kindly merge PR.
|
Kind ask to merge PR |
Updates yargs to address high severity security vulnerability as detailed in #252
Note
there are two failing tests, these already exist on the master branch