-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
devDeps: squirrelly 9 #19113
devDeps: squirrelly 9 #19113
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
56a23a7
to
d678363
Compare
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
|
96357c2
to
517ad20
Compare
Codecov Report
@@ Coverage Diff @@
## develop #19113 +/- ##
===========================================
- Coverage 70.09% 70.08% -0.01%
===========================================
Files 963 963
Lines 37222 37222
Branches 9622 9622
===========================================
- Hits 26089 26087 -2
- Misses 11133 11135 +2 |
dad444a
to
2463a34
Compare
afbaf35
to
bd2615e
Compare
bd2615e
to
fed1310
Compare
@kumavis ci e2e timeout errors look unrelated but merging is blockeddue to not all checks passing. Should I do a no-diff force-push to retrigger the flakey test? EDIT: Had to do it anyway due to merge-conflict. Nothing should be changed here from last review round. |
fed1310
to
fb81d7d
Compare
47b4932
to
d86d9bb
Compare
Explanation
CVE-2021-32819 / GHSA-q8j6-pwqx-pm96
Change (guess who 😉): squirrellyjs/squirrelly#254
v9 release notes: https://github.com/squirrellyjs/squirrelly/releases/tag/v9.0.0
Testing instructions
TODO:
Pre-merge author checklist
Pre-merge reviewer checklist