-
-
Notifications
You must be signed in to change notification settings - Fork 344
chore(deps): v7: Bump Android SDK to v8.9.0 #4775
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
Conversation
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 🚀 Ty!
Android (legacy) Performance metrics 🚀
|
Android (new) Performance metrics 🚀
|
iOS (new) Performance metrics 🚀
|
CHANGELOG.md
Outdated
- Bump Android SDK from v7.20.1 to v8.8.0 ([#4490](https://github.com/getsentry/sentry-react-native/pull/4490), [#4663](https://github.com/getsentry/sentry-react-native/pull/4663), [#4755](https://github.com/getsentry/sentry-react-native/pull/4755)) | ||
- [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#880) | ||
- [diff](https://github.com/getsentry/sentry-java/compare/7.20.1...8.8.0) | ||
- Bump Android SDK from v7.20.1 to v8.9.0 ([#4490](https://github.com/getsentry/sentry-react-native/pull/4490), [#4663](https://github.com/getsentry/sentry-react-native/pull/4663), [#4755](https://github.com/getsentry/sentry-react-native/pull/4755), [#4775](https://github.com/getsentry/sentry-react-native/pull/4775)) |
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.
This list is getting quite large, shouldn't we keep the main PR with the main changes and only keep the latest PR that only bumped the version number?
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.
This makes sense @lucas-zimerman 👍 Updated with c69debe
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.
left a small note, but other than that lgtm!
iOS (legacy) Performance metrics 🚀
|
📢 Type of change
📜 Description
Bumps Android SDK from
8.8.0
to8.9.0
targeting v7💡 Motivation and Context
💚 How did you test it?
Manual, CI
📝 Checklist
sendDefaultPII
is enabled🔮 Next steps