Skip to content

[cli] Fix error message for launching updates on iOS physical devices#348

Merged
gabrieldonadel merged 1 commit into
expo:mainfrom
patrickwehbe:cli-fix-ios-launch-update-error-message
Jun 20, 2026
Merged

[cli] Fix error message for launching updates on iOS physical devices#348
gabrieldonadel merged 1 commit into
expo:mainfrom
patrickwehbe:cli-fix-ios-launch-update-error-message

Conversation

@patrickwehbe

Copy link
Copy Markdown
Contributor

Why

The error thrown when launching an update on an iOS physical device is missing a word. In launchUpdateOnIOSAsync it currently reads:

Launching updates on iOS physical is not supported yet

The equivalent guard in launchUpdateOnExpoGoIosAsync (same file, line 115) already has the correct wording:

Launching updates on iOS physical devices is not supported yet

Both run the same !isSimulator check, so the two messages should match.

How

Added the missing word "devices" to the error string on line 151 of apps/cli/src/commands/LaunchUpdate.ts so it matches its sibling on line 115. Also added a changelog entry under the Unpublished bug fixes section.

Test Plan

This is a one-word change to a string literal inside an existing throw new Error(...). The two error messages in the file are now identical, which can be confirmed by searching the file for "Launching updates on iOS physical" and seeing both lines read the same.

The error thrown in launchUpdateOnIOSAsync was missing the word "devices", so it read "Launching updates on iOS physical is not supported yet". The matching guard in launchUpdateOnExpoGoIosAsync already reads "...iOS physical devices is not supported yet". Align the two messages.
@patrickwehbe
patrickwehbe force-pushed the cli-fix-ios-launch-update-error-message branch from c8c2790 to 94c2d30 Compare June 20, 2026 11:13

@gabrieldonadel gabrieldonadel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@patrickwehbe thanks for helping with this! 👏

@gabrieldonadel
gabrieldonadel merged commit b241e3b into expo:main Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants