-
-
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 plist
resolution
#8835
Conversation
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/plist@3.0.5 |
c983f03
to
ee3a437
Compare
Bitrise✅✅✅ Commit hash: ee3a437 Note
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8835 +/- ##
=======================================
Coverage 41.61% 41.61%
=======================================
Files 1269 1269
Lines 30869 30869
Branches 3086 3086
=======================================
Hits 12847 12847
Misses 17245 17245
Partials 777 777 ☔ View full report in Codecov by Sentry. |
ee3a437
to
a13142d
Compare
The resolution for `plist` was added in #3987 to resolve a security advisory. At the time, the resolution was used to update the package, though a resolution wasn't needed to do this in the first place. Now that the package has been updated, this resolution is only having the effect of preventing further updates.
a13142d
to
8add793
Compare
Quality Gate passedIssues Measures |
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
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 me.
Description
The resolution for
plist
was added in #3987 to resolve a security advisory. At the time, the resolution was used to update the package, though a resolution wasn't needed to do this in the first place. Now that the package has been updated, this resolution is only having the effect of preventing further updates.Related issues
N/A
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist