Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add minimalist .xcodeproj so rnpm finds DeviceUtil.{h,m} #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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