-
Notifications
You must be signed in to change notification settings - Fork 191
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
security(deps): bump filenamify-url to 2.1.1 #393
Conversation
regenerated lock file from scratch to get back to 0 vulnerabilities
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.
For what it's worth, the only breaking change in filenamify-url v2 is that it requires Node.js 8, so no other code changes should be necessary. 👍🏼
Though I'm not sure that regenerating the lockfile was necessary— |
There were other audit failures from locked deps. |
Related issue: sindresorhus/filenamify-url#9 |
heya @tschaub, any chance you've got time to review this? :) |
Thanks, @AviVahl. There are automated security updates configured for this repo, but they can take up to 7 days from the time of an alert. The alert for |
I think this change broke our ci. we clone our repo via something like:
our ci job now fails during a
|
regenerated lock file from scratch to get back to 0 vulnerabilities
fixes: