-
-
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 ua-parser-js
resolution
#8816
Conversation
Bitrise✅✅✅ Commit hash: 564986d Note
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8816 +/- ##
=======================================
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. |
3ca8578
to
b9e4d3d
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
This resolution was added in #5594 to force the package to update, but the resolution was not actually needed. Effectively it serves only to downgrade the v1 version of this package, which is not necessary.
b9e4d3d
to
ba48aa6
Compare
Quality Gate passedIssues Measures |
Description
This resolution was added in #5594 to force the package to update, but the resolution was not actually needed. Effectively it serves only to downgrade the v1 version of this package, which is not necessary.
Related issues
N/A
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist