Skip to content

Conversation

@cipolleschi
Copy link
Contributor

Summary:
This Diff aligns the way in which iOS and Android codegen the modules and components.
Android takes all the JS in the project root folder and creates components starting from there.
iOS used to required to specify a specific path for each component, within a JSON field called libraries. This Diff let iOS work in the same way as android does

Backward compatibility: This diff still support the old way for iOS, but we are deprecating it.

Changelog

[iOS][Added] - Support codegen from a single folder

Differential Revision: D36473005

@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 fb-exported labels May 18, 2022
@facebook-github-bot
Copy link
Contributor

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

@react-native-bot react-native-bot added Platform: iOS iOS applications. Type: Enhancement A new feature or enhancement of an existing feature. labels May 18, 2022
@analysis-bot
Copy link

analysis-bot commented May 18, 2022

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,782,995 +0
android hermes armeabi-v7a 7,186,489 +0
android hermes x86 8,094,638 +0
android hermes x86_64 8,072,076 +0
android jsc arm64-v8a 9,655,567 +0
android jsc armeabi-v7a 8,427,832 +0
android jsc x86 9,607,811 +0
android jsc x86_64 10,202,443 +0

Base commit: c19bc0b
Branch: main

@analysis-bot
Copy link

analysis-bot commented May 18, 2022

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: c19bc0b
Branch: main

Summary:
Pull Request resolved: facebook#33864

This Diff aligns the way in which iOS and Android codegen the modules and components.
Android takes all the JS in the project root folder and creates components starting from there.
iOS used to required to specify a specific path for each component, within a JSON field called `libraries`. This Diff let iOS work in the same way as android does

**Backward compatibility:** This diff still support the old way for iOS, but we are deprecating it.

## Changelog
[iOS][Added] - Support codegen from a single folder

Reviewed By: cortinico

Differential Revision: D36473005

fbshipit-source-id: f752f1e80cf95fa567514b1cd7c24c42a052b0a6
@facebook-github-bot
Copy link
Contributor

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

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @cipolleschi in 05aaba9.

When will my fix make it into a release? | Upcoming Releases

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 Merged This PR has been merged. p: Facebook Partner: Facebook Partner Platform: iOS iOS applications. Type: Enhancement A new feature or enhancement of an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants