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

[RN][Testing] Update testing scripts to work with any version of React native #45201

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

cipolleschi
Copy link
Contributor

@cipolleschi cipolleschi commented Jun 27, 2024

Summary:

These changes updates the testing script so that it follow the approach described here.

In this way, the testing script:

  • Is not bound to a single version of React Native
  • Does not require that we publish an rc.X version of the template before we release rc.X of React Native

Changelog:

[Internal] - Update testing script to work with any version of React Native

Test Plan:

Tested locally and verified that the script is packing a template and using the template when initializing the project.

Screenshot 2024-06-27 at 11 47 03

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 27, 2024
@cipolleschi cipolleschi changed the base branch from main to 0.75-stable June 27, 2024 09:43
@cipolleschi cipolleschi force-pushed the cipolleschi/update-testing-scripts branch from f670fbf to dbdb477 Compare June 27, 2024 09:54
@cipolleschi cipolleschi marked this pull request as ready for review June 27, 2024 09:54
@analysis-bot
Copy link

analysis-bot commented Jun 27, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 20,443,812 +142,913
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 23,642,505 +145,011
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 8db2995
Branch: main

@cortinico cortinico force-pushed the cipolleschi/update-testing-scripts branch from dbdb477 to b03d3f6 Compare July 3, 2024 12:41
@cortinico
Copy link
Contributor

I tested this locally on the 0.75-branch and I was able to successfully e2e test the template 🎉

@@ -59,6 +61,7 @@ async function main() {

Options:
--projectName The name of the new React Native project.
--currentBranch The current branch to checkout.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: this is a bit unclear. We should clarify that you're referring to the branch of the template repo

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. p: Facebook Partner: Facebook Partner Pick Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants