meteorrn / meteor-react-native

Meteor client for React Native matching Meteor Spec
https://guide.meteor.com/react-native.html
Other
59 stars 31 forks source link

Make wolfy87-eventemitter an optional dependency #112

Closed jankapunkt closed 1 year ago

jankapunkt commented 1 year ago

As stated in the code:

// TODO: 0.60+ supports import { EventEmitter } from 'events';

Therefore we might rather make this an optional dependency and update the docs. RN > 60 should be very common by now.

TheRealNate commented 1 year ago

@jankapunkt For compatibility we already say that RN <0.60 isn't supported, so I don't think we even need to make it optional, we can just fully move over.

github-actions[bot] commented 1 year ago

Closing this issue due to no activity. Feel free to reopen.

jankapunkt commented 1 year ago

keep open

jankapunkt commented 1 year ago

Implemented in 2.5.0

get it via https://www.npmjs.com/package/@meteorrn/core/v/2.5.0