-
Notifications
You must be signed in to change notification settings - Fork 27k
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
Upgrade browserslist to 4.16.6 #24692
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…ded" This reverts commit 64f3868.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Thank you @kachkaev for working on this. We also got this raised in Veracode tool Software Composition Analysis report 👍 |
This comment has been minimized.
This comment has been minimized.
Hey folks! Do I need to change anything in this PR? The CI is failing, but that seems to be a common case, even for the just-merged PRs. Happy to make further enhancements this weekend! |
Thank you @kachkaev following up on this, I hope you'll be able to get this merged 👍 |
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.
Looks good to land now, thanks 👍
## Bug - [x] Related issue: [CVE-2021-23364](https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-23364) - ~Integration tests added~ (N/A)
Bug
Integration tests added(N/A)