expo / sentry-expo

MIT License
202 stars 83 forks source link

Upgrade to Sentry React Native V4 #266

Closed kbrandwijk closed 2 years ago

kbrandwijk commented 2 years ago

Tested the config plugin and module locally (installed on a project, configured Sentry and observed exceptions coming in, used config plugin with expo publish). Also made a few minor changes to the tests that matched the proposed changes in another PR.

As far as I'm concerned, there are no breaking changes, but it looks like this has to be released as a major release, because of the liberal semver range in the versions endpoint (^3.1.1). This is because @sentry/react-native is installed manually by developers in their project, and if this lib updates, it will break if users don't manually update @sentry/react-native to v4, so we don't want this to be automatically updated to.

SimenB commented 2 years ago

@kbrandwijk could you update https://github.com/expo/sentry-expo/blob/f8db1c312e2faf7f580f5879ccd8f5df1009eb11/package.json#L40 to v5 as well?