-
Notifications
You must be signed in to change notification settings - Fork 85
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
Update @segment dependencies #2598
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2598 +/- ##
==========================================
- Coverage 79.54% 79.53% -0.01%
==========================================
Files 592 592
Lines 21272 21272
Branches 3884 3884
==========================================
- Hits 16920 16918 -2
- Misses 4302 4304 +2
Partials 50 50
Continue to review full report at Codecov.
|
@jeanregisser is there a good way to ensure that this update is using >=com.segment.analytics.android:analytics:4.10.1? |
@MuckT you can run |
@jeanregisser thanks for the info! It looks like this puts our com.segment.analytics.android:analytics:4.10.4. There are some other dependencies that got updated; however, if there are no issues I don't see a downside to including these also. |
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.
Shipit! 🚢
Hey @MuckT since NA is mentioned in how others should test , we are skipping this PR |
Description
Updates segment related dependencies upgrade from
com.segment.analytics.android:analytics:4.9.3
tocom.segment.analytics.android:analytics:4.10.1
for Google PlayStore Compliance.Other changes
N/A
Tested
Tested Locally on Android.
How others should test
This does not need to be tested by QA.
Related issues
N/A
Backwards compatibility
Yes