Skip to content
This repository has been archived by the owner on Apr 2, 2021. It is now read-only.

Remove some files from the repo, use files instead of .npmignore #551

Merged
merged 1 commit into from
Jun 14, 2019

Conversation

janicduplessis
Copy link
Contributor

Those things were probably useful when this was in fb internal repo but we can remove them now.

@janicduplessis janicduplessis merged commit 71053fe into facebookarchive:master Jun 14, 2019
@dozoisch
Copy link
Contributor

Hi @janicduplessis ! Just noticed that this PR removed ios_setup.js but this is the way mentionned in the official documentation.

https://developers.facebook.com/docs/react-native/configure-ios

image

@janicduplessis
Copy link
Contributor Author

@dozoisch Thanks for pointing it out, I'll see if I can get this doc updated. The installation instructions here should work. However I think we should update the instruction to recommend using CocoaPods.

@dozoisch
Copy link
Contributor

@janicduplessis perfect thanks!

@hugoh59
Copy link

hugoh59 commented Jul 11, 2019

I'm having this issue as well.

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.

4 participants