Skip to content
New issue

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

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

Already on GitHub? # to your account

Remove optional codegen config inside template #33108

Closed
wants to merge 1 commit into from

Conversation

ShikaSD
Copy link
Contributor

@ShikaSD ShikaSD commented Feb 14, 2022

Summary:
The codegen config is optional and can be removed from the default package.json configuration in the template to simplify 0.68 upgrade for people who are not opted-in to the new arch.

Changelog: [Internal] - Remove optional codegenConfig field from template

Differential Revision: D34216988

Summary:
The codegen config is optional and can be removed from the default package.json configuration in the template to simplify 0.68 upgrade for people who are not opted-in to the new arch.

Changelog: [Internal] - Remove optional codegenConfig field from template

Differential Revision: D34216988

fbshipit-source-id: e7deb11ffced4df03c20e4b00e009948d138ba99
@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 Feb 14, 2022
@facebook-github-bot
Copy link
Contributor

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

@analysis-bot
Copy link

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

Base commit: f89a0b7
Branch: main

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,189,108 +0
android hermes armeabi-v7a 7,789,539 +0
android hermes x86 8,559,204 +0
android hermes x86_64 8,511,588 +0
android jsc arm64-v8a 9,857,605 +0
android jsc armeabi-v7a 8,842,911 +0
android jsc x86 9,824,292 +0
android jsc x86_64 10,420,764 +0

Base commit: f89a0b7
Branch: main

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @ShikaSD in a86c521.

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

@react-native-bot react-native-bot added the Merged This PR has been merged. label Feb 15, 2022
ShikaSD pushed a commit that referenced this pull request Feb 24, 2022
Summary:
Pull Request resolved: #33108

The codegen config is optional and can be removed from the default package.json configuration in the template to simplify 0.68 upgrade for people who are not opted-in to the new arch.

Changelog: [Internal] - Remove optional codegenConfig field from template

Reviewed By: cortinico

Differential Revision: D34216988

fbshipit-source-id: 5c448472eed99bc112aef204c4025454171a83c5
ShikaSD pushed a commit that referenced this pull request Feb 24, 2022
Summary:
Pull Request resolved: #33108

The codegen config is optional and can be removed from the default package.json configuration in the template to simplify 0.68 upgrade for people who are not opted-in to the new arch.

Changelog: [Internal] - Remove optional codegenConfig field from template

Reviewed By: cortinico

Differential Revision: D34216988

fbshipit-source-id: 5c448472eed99bc112aef204c4025454171a83c5
leotm added a commit to leotm/react-native-template-new-architecture that referenced this pull request Feb 25, 2022
- Update app new arch Gradle build config
- Add RubyGems lockfile (check why missing online: upgrade-helper/?from=0.68.0-rc.0&to=0.68.0-rc.1)
- Keep: facebook/react-native#33108 for codegen config later

├─┬ react-native-gradle-plugin@0.0.5
│ └── react-native-codegen@0.0.13
├─┬ react-native-screens@3.12.0
│ └─┬ react-native-gradle-plugin@0.0.3
│   └── react-native-codegen@0.0.13
└─┬ react-native@0.68.0-rc.2
  └── react-native-codegen@0.0.13
Saadnajmi pushed a commit to Saadnajmi/react-native-macos that referenced this pull request Jan 15, 2023
Summary:
Pull Request resolved: facebook#33108

The codegen config is optional and can be removed from the default package.json configuration in the template to simplify 0.68 upgrade for people who are not opted-in to the new arch.

Changelog: [Internal] - Remove optional codegenConfig field from template

Reviewed By: cortinico

Differential Revision: D34216988

fbshipit-source-id: 5c448472eed99bc112aef204c4025454171a83c5
# for free to join this conversation on GitHub. Already have an account? # 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants