Skip to content

Conversation

@huntie
Copy link
Member

@huntie huntie commented Mar 1, 2024

Summary:
Addresses a gap when using the set-version script to update all packages on main (i.e. post branch cut):

  • Package versions were not being set consistently. It is safe to version all workspace packages, including "private".
    • Our publishing workflow is independent from this, and only considers public packages for submission to npm.
  • We also need to update the root package.json, which includes devDependencies referencing workspace dependencies.

Unblocks #43132.

Changelog: [Internal]

Differential Revision: D54419456

huntie added 2 commits March 1, 2024 08:21
Summary:
Make the snapshot output of this test terser (since `set-version` is a superset of the fully tested `set-rn-version` script). Notably, this removes any instances of `generated` from the snapshot file, which would hide the diff in PRs.

Changelog: [Internal]

Differential Revision: D54420338
Summary:
Addresses a gap when using the `set-version` script to update all packages on `main` (i.e. post branch cut):
- Package versions were not being set consistently. It is safe to version all workspace packages, including `"private"`.
    - Our publishing workflow is independent from this, and only considers public packages for submission to npm.
- We also need to update the root `package.json`, which includes `devDependencies` referencing workspace dependencies.

Unblocks facebook#43132.

Changelog: [Internal]

Differential Revision: D54419456
@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 1, 2024
@facebook-github-bot
Copy link
Contributor

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

@huntie huntie closed this Mar 5, 2024
@huntie huntie deleted the export-D54419456 branch September 9, 2024 12:02
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.

2 participants