Skip to content

Conversation

@peritus
Copy link

@peritus peritus commented Jan 27, 2016

Fixes #23

@dannolan
Copy link

dannolan commented Apr 4, 2016

+1

@GertjanReynaert
Copy link
Owner

@peritus Is this pr still valid for the current React Native version?

@peritus
Copy link
Author

peritus commented Apr 4, 2016

@GertjanReynaert I have no idea. I can rebase and test, there shouldn't be any dependencies on the react-native version, if you ask me.

so rnpm finds DeviceUtil.{h,m}
@GertjanReynaert
Copy link
Owner

@peritus I'm just asking because I'm not that familiar anymore with React Native.
Since adding RNPM support is a pretty frequently requested feature, and this would enable RNPM, I would like to get this merged.

So if you could just validate that this works, then I'll merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants