Skip to content

Relocate and refactor set-rn-version script#42730

Closed
huntie wants to merge 1 commit into
facebook:mainfrom
huntie:export-D53001971
Closed

Relocate and refactor set-rn-version script#42730
huntie wants to merge 1 commit into
facebook:mainfrom
huntie:export-D53001971

Conversation

@huntie
Copy link
Copy Markdown
Member

@huntie huntie commented Jan 30, 2024

Summary:
Reviewing and modernising this script as part of simplifying our release publish workflow.

  • Drop unused --dependency-versions arg from CLI entry point
  • Simplify templating approach
  • Type as Flow
  • Drop dependencies on shelljs and yargs
  • Relocate under scripts/releases/
  • Rewrite tests as snapshot tests

Changelog: [Internal]

Reviewed By: NickGerleman

Differential Revision: D53001971

@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 Jan 30, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

Summary:
Pull Request resolved: facebook#42730

Reviewing and modernising this script as part of simplifying our release publish workflow.

- Drop unused `--dependency-versions` arg from CLI entry point
- Simplify templating approach
- Type as Flow
- Drop dependencies on `shelljs` and `yargs`
- Relocate under `scripts/releases/`
- Rewrite tests as snapshot tests

Changelog: [Internal]

Reviewed By: NickGerleman

Differential Revision: D53001971

fbshipit-source-id: 34534a0fcbcf70832101105f4d18246c076308c8
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in e97ba47.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants