-
-
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 pac-resolver
resolution
#8811
Conversation
b3bdf68
to
92c10a2
Compare
pac-resolver
resolutionpac-resolver
resolution
Bitrise✅✅✅ Commit hash: 92c10a2 Note
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8811 +/- ##
=======================================
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. |
92c10a2
to
a68cdfc
Compare
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/pac-resolver@5.0.1 |
This resolution was added in #3280 to force this package to update, to resolve a security advisory. However, the package using this package has since been updated, and this resolution is now having the effect of _downgrading_ this package from v7 to v5.
a68cdfc
to
b7321a9
Compare
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
Quality Gate passedIssues Measures |
Description
This resolution was added in #3280 to force this package to update, to resolve a security advisory. However, the package using this package has since been updated, and this resolution is now having the effect of downgrading this package from v7 to v5.
Related issues
N/A
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist