Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

WIP - Update sentry take 2 #2095

Closed
wants to merge 4 commits into from
Closed

WIP - Update sentry take 2 #2095

wants to merge 4 commits into from

Conversation

brainbicycle
Copy link
Contributor

Mirroring @ds300's comment from old PR here:

Need to run some more testing on this

  • make sure native crashes are reported correctly
  • make parallel branch for eigen
  • understand and test eigen-specific crash reporting code (breadcrumbs etc?)
  • investigate unhandled promise rejection reports (doesn't work in dev mode, check in a beta build if possible)

Remove react-native-sentry

Add @sentry/react-native

Update npm podspecs with new sentry

Update sentry in podspec
Update sentry imports and usage

Update sentry mocks in tests

Remove old sentry warning ignore in app registry

Remove old sentry patch file
Update Sentry pod in example app, see details

Update sentry pod in example app

Update sentry imports and usage in example app

Update missed Sentry usage
Update yarn cache version to fix ci issues

Disable travis npm :wqcache to fix build issues
@brainbicycle
Copy link
Contributor Author

Closing and reopening in combined repo.
artsy/eigen#3037

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant