heap / react-native-heap

A React Native integration for Heap
MIT License
82 stars 36 forks source link

[HEAP-45444] Resolved critical warnings and got unit tests working #383

Closed bnickel closed 1 year ago

bnickel commented 1 year ago

HEAP-45444

The primary change in this PR is the removal of the @expo/config-plugins@^4.1.5 dependency. This dependency wasn't needed because Expo already imports @expo/config-plugins as one of its dependencies and if you aren't using Expo you don't need to bring in the stack. It's not even worth calling for a peer dependency in this case.

While working through that, I decided to clear up any critical warnings and build issues in the SDK so we can run the following commands without issue:

That resulted in the following changes:

Test Plan

Checklist