-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chore: Remove nanoid
resolution
#8809
Conversation
ddf873a
to
dd41b8e
Compare
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/nanoid@3.2.0 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8809 +/- ##
=======================================
Coverage 41.60% 41.60%
=======================================
Files 1269 1269
Lines 30859 30859
Branches 3082 3082
=======================================
Hits 12840 12840
Misses 17243 17243
Partials 776 776 ☔ View full report in Codecov by Sentry. |
Bitrise✅✅✅ Commit hash: bf0073f Note
|
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.
LGTM
This resolution for `nanoid` was added in #3641 to resolve a security advisory. The resolution is not needed, as the advisory is not present after removing the resolution. This resolution was unsafe because it was forcing the use of incompatible versions of the package.
bf0073f
to
e962ff9
Compare
Quality Gate passedIssues Measures |
Description
This resolution for
nanoid
was added in #3641 to resolve a security advisory. The resolution is not needed, as the advisory is not present after removing the resolution.This resolution was unsafe because it was forcing the use of incompatible versions of the package.
Related issues
N/A
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist