expo / sentry-expo

MIT License

203 stars 83 forks source link

chore: replace `@sentry/browser` with `@sentry/react` #300

Closed SimenB closed 1 year ago

SimenB commented 1 year ago

Checklist

[x] I've added an entry to CHANGELOG.md if necessary.

Why

This is the same module @sentry/react-native pulls in, so makes sense I think.

https://npmjs.com/package/@sentry/react

This package is a wrapper around @sentry/browser, with added functionality related to React. All methods available in @sentry/browser can be imported from @sentry/react.

How

CHanged the dep and imports

Test Plan

🤷

kbrandwijk commented 1 year ago

Thank you very much for your contribution. I've wrapped these changes (with a slight variation) into #302, as those are breaking changes and I'd rather ship a single major release.