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

Connect ReactNativeDependencies with ios jobs #49978

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

cipolleschi
Copy link
Contributor

Summary:
This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

Changelog:

[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Mar 12, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

…e folly flags (facebook#49969)

Summary:
Pull Request resolved: facebook#49969

Improved RCT-Folly podspec by configuring it as a prepare step, so that we don't need to forward the folly flags to all the others pods and to 3rd party dependencies.

## Changelog:

[INTERNAL] - Configure folly in a prepare step

Pull Request resolved: facebook#49812

Test Plan:
✅ Run Rn-Tester and  verify that it works as expected both building deps from source and using prebuilt tarballs
✅ Add third-party library (react-native-reanimated) and perform the same tests to verify that it works with the changed podspec and utilities

Differential Revision: D71032640

Reviewed By: javache

Pulled By: javache
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
…ok#49812)

Summary:
There are two environment variables that is related to ReactNativeDependencies:
- `RCT_USE_DEP_PREBUILD `: If set to 1, ReactNativeDependencies will be built from source.
- `RCT_DEPS_VERSION`: If set to 1, it will override the version of ReactNativeDependencies to be used.
bypass-github-export-checks
## Changelog:

[INTERNAL] - Introduced functions to configure ReactNativeDependencies in Cocoapods

Pull Request resolved: facebook#49812

Test Plan:
✅ Run Rn-Tester and  verify that it works as expected both building deps from source and using prebuilt tarballs
✅ Add third-party library (react-native-reanimated) and perform the same tests to verify that it works with the changed podspec and utilities

Differential Revision: D70968672

Reviewed By: javache

Pulled By: javache
…cies.podspec (facebook#49971)

Summary:
Pull Request resolved: facebook#49971

Updated the Cocoapods infrastructure to use the new  the `ReactNativeDependencies.podspec`

## Changelog:

[INTERNAL] - Configured the Cocoapods infra to use the new `ReactNativeDependencies.podspec`

Test Plan:
✅ Run Rn-Tester and  verify that it works as expected both building deps from source and using prebuilt tarballs
✅ Add third-party library (react-native-reanimated) and perform the same tests to verify that it works with the changed podspec and utilities

Differential Revision: D71032638

Reviewed By: javache

Pulled By: javache
…ce (facebook#49968)

Summary:
Pull Request resolved: facebook#49968

Updates all the core podspec to use the new `ReactNativeDependencies` podspec

## Changelog:

[INTERNAL] - Updates Modules in core to use the new dependencies.

Test Plan:
✅ Run Rn-Tester and  verify that it works as expected both building deps from source and using prebuilt tarballs
✅ Add third-party library (react-native-reanimated) and perform the same tests to verify that it works with the changed podspec and utilities

Differential Revision: D71032639
)

Summary:
Pull Request resolved: facebook#49970

This change add supports to pass a tarball from a local file to the ReactNativeDependencies podspec, so that we can build React Native using a local copy or the RNDependencies and we can use it also in CI.

## Changelog:

[INTERNAL] - Add support for local tarballs

Differential Revision: D71032641
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Reviewed By: cortinico

Differential Revision: D71034587
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71034587

cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Mar 12, 2025
Summary:
Pull Request resolved: facebook#49978

This change connects the RNDependencies we prebuild in CI with the other iOS jobs we have.

By doing so, we aim to speed up the build time in CI, saving time and money.

## Changelog:
[Internal] - Connect the ReactNativeDependencies in CI

Differential Revision: D71034587
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants