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

[Headers] Add an umbrella header for the library #261

Closed
wants to merge 1 commit into from

Conversation

ide
Copy link
Contributor

@ide ide commented Mar 26, 2015

The umbrella header allows projects that link React Native using CocoaPods to import it in standard fashion: #import <React/React.h>. Currently all of the headers are exported so that external projects can experiment.

@sahrens
Copy link
Contributor

sahrens commented Mar 31, 2015

cc @nicklockwood

@sahrens
Copy link
Contributor

sahrens commented Mar 31, 2015

cc @tadeuzagallo too.

@ide
Copy link
Contributor Author

ide commented Mar 31, 2015

This has been kind of hacked together as React Native has been refactored, so I think the concept stands but each library should have its own umbrella header, and then maybe have a master umbrella header that imports the core + ubiquitous libraries (text, image support but not geolocation, vibration, etc).

@tadeuzagallo
Copy link
Contributor

Sounds good. Everything that is non-optional is exposed in this header, and one more header per subspec.

@ide ide force-pushed the umbrella-header branch from 78a49d0 to dd1b5ec Compare March 31, 2015 19:53
The umbrella header allows projects that link React Native using CocoaPods to import it in standard fashion: `#import <React/React.h>`. Currently all of the headers are exported so that external projects can experiment.

Generated with:

    find React -name "*.h" | awk -F'/' '{print "#import <React/"$NF">"}' > React/React.h
@ide ide force-pushed the umbrella-header branch from dd1b5ec to 5130895 Compare March 31, 2015 23:57
@ide ide closed this Apr 1, 2015
@ide ide deleted the umbrella-header branch April 1, 2015 06:44
@ide ide restored the umbrella-header branch April 1, 2015 06:44
facebook-github-bot pushed a commit that referenced this pull request Jan 17, 2017
Summary:
Fixes #261
Closes facebook/yoga#262

Reviewed By: splhack

Differential Revision: D4245200

Pulled By: emilsjolander

fbshipit-source-id: 77d802d71010ed426511d6a01e6de1e7c9194179
nicktate pushed a commit to nicktate/react-native that referenced this pull request Jan 19, 2017
Summary:
Fixes facebook#261
Closes facebook/yoga#262

Reviewed By: splhack

Differential Revision: D4245200

Pulled By: emilsjolander

fbshipit-source-id: 77d802d71010ed426511d6a01e6de1e7c9194179
mganandraj pushed a commit to mganandraj/react-native that referenced this pull request Mar 26, 2020
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
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