Skip to content

fix: version checker not considering nightlies#43712

Closed
tido64 wants to merge 1 commit into
mainfrom
tido/fix-latest-version-check
Closed

fix: version checker not considering nightlies#43712
tido64 wants to merge 1 commit into
mainfrom
tido/fix-latest-version-check

Conversation

@tido64

@tido64 tido64 commented Mar 29, 2024

Copy link
Copy Markdown
Collaborator

Summary:

Fix version checker not considering nightlies:

  WARNING: You should run npx react-native@latest to ensure you're always using the most current version of the CLI. NPX
 has cached version (0.74.0-nightly-20240214-b8ad91732) != current release (0.73.6)

Changelog:

[GENERAL] [FIXED] - Fix version checker not considering nightlies

Test Plan:

On a recent nightly version, run any cli command.

@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: Microsoft Partner: Microsoft Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Mar 29, 2024
@analysis-bot

Copy link
Copy Markdown
Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 18,876,971 +5
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 22,250,018 -2
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: d0cac87
Branch: main

@tido64 tido64 requested a review from blakef March 29, 2024 09:21
Comment thread packages/react-native/cli.js
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@zeyap has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@zeyap merged this pull request in 489d9f6.

@tido64 tido64 deleted the tido/fix-latest-version-check branch April 2, 2024 05:53
zeyap added a commit to zeyap/react-native that referenced this pull request Apr 2, 2024
…book#43744)

Summary:

## Changelog:
Use semver to do version comparison in cli instead of custom fn `parseVersion`

~~adding test for PR change in facebook#43712

Differential Revision: D55600458
facebook-github-bot pushed a commit that referenced this pull request Apr 9, 2024
Summary:
Pull Request resolved: #43744

## Changelog:
[General] [Changed] - Use semver to do version comparison in cli instead of custom fn `parseVersion`

~~adding test for PR change in #43712

Reviewed By: NickGerleman

Differential Revision: D55600458

fbshipit-source-id: eaffa3e89696b90fd354aefaba79e035d180a0b5
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. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants