Skip to content
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

upgrade @sentry/react-native to 5.9.1 #367

Closed
wants to merge 1 commit into from

Conversation

mroswald
Copy link

@mroswald mroswald commented Sep 5, 2023

Checklist

Why

Sentry offers profiling in beta state since version 5.8.0 of @sentry/react-native. Upgrading to latest 5.9.1

How

Updated the package and peer versions accordingly to @sentry/react-native.

Test Plan

Used an expo app and included the new version. Sentry log shows 5.9.1

package.json Outdated
"@sentry/types": "7.52.1",
"@sentry/integrations": "7.60.1",
"@sentry/react": "7.60.1",
"@sentry/react-native": "5.8.0",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we upgrade all the way to 5.9.1?

@jeffreysilver
Copy link

I'm hoping to get sentry profiling up and running on an Expo app as well - would be great to get this merged in @kbrandwijk!

@mroswald
Copy link
Author

mroswald commented Sep 6, 2023

I'm hoping to get sentry profiling up and running on an Expo app as well - would be great to get this merged in @kbrandwijk!

Done, but needs some testing

@mroswald mroswald changed the title upgrade @sentry/react-native to 5.8.0 upgrade @sentry/react-native to 5.9.1 Sep 6, 2023
@ghost
Copy link

ghost commented Sep 20, 2023

Any plans to merge this soon?

@mroswald
Copy link
Author

mroswald commented Sep 28, 2023

Tested with 5.9.1 and it works. Build on XCode 15 with iOS17. Also rebased to current main branch

@mroswald mroswald force-pushed the upgrade-sentry-react-native-580 branch from 33f65c5 to 24e7dda Compare September 28, 2023 20:32
@jeffreysilver
Copy link

jeffreysilver commented Sep 29, 2023

Any plans to merge this in? We're excited to get profiling up and running to help debug some performance issues

@mroswald
Copy link
Author

Seems it was updated already in the meantime.. will close this PR

@mroswald mroswald closed this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants