Skip to content

Update test-e2e-local to use source monorepo packages for RNTestProject - #42899

Closed
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D53484510
Closed

Update test-e2e-local to use source monorepo packages for RNTestProject#42899
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D53484510

Conversation

@huntie

@huntiehuntie commented Feb 6, 2024

Copy link
Copy Markdown
Collaborator

Summary:

Updates the test-e2e-local script to bootstrap /tmp/RNTestProject/ using the currently checked out repository as the source of truth for all monorepo packages (previously we only did this for the react-native package).

This enables release testers to validate a release before physically publishing new dependency versions via yarn bump-all-updated-packages.

We are able to reuse the scripts/template/initialize.js script that is currently used for E2E validation in CI. This sets up a local Verdaccio server during project install.

Changelog: [Internal]

Differential Revision: D53484510

@facebook-github-botfacebook-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 Feb 6, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@analysis-bot

analysis-bot commented Feb 6, 2024

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a17,828,330+3
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a21,186,796+4
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 1570062
Branch: main

@huntiehuntie changed the title Update test-e2e-local to create RNTestProject using in-repo packagesUpdate test-e2e-local to use source monorepo packages for RNTestProjectFeb 9, 2024
@huntie
huntieforce-pushed the export-D53484510 branch 2 times, most recently from 69c31e8 to a2cb3faCompareFebruary 13, 2024 16:58
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

…ct (#42899)
Summary:
Updates the `test-e2e-local` script to bootstrap `/tmp/RNTestProject/` using the currently checked out repository as the source of truth for all monorepo packages (previously we only did this for the `react-native` package).
This enables release testers to validate a release **before** physically publishing new dependency versions via `yarn bump-all-updated-packages`.
We are able to reuse the `scripts/template/initialize.js` script that is currently used for E2E validation in CI. This sets up a local Verdaccio server during project install.
NOTE: The time taken for `Build packages` + Verdaccio isn't ideal, I may explore a way to reuse the published package state in a future diff. Until then, this extra time (~1 min) will still be much less pain than the `bump-all-updated-packages` + commit process loop.
Changelog:
[Internal] - Update test-e2e-local to use source monorepo packages for RNTestProject
Reviewed By: lunaleaps
Differential Revision: D53484510
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Feb 15, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in e4135e9.

@huntie
huntie deleted the export-D53484510 branch February 16, 2024 15:06
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@huntie@facebook-github-bot@analysis-bot