Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cli): fix npx expo start --dev-client --ios #18747

Merged
merged 2 commits into from Aug 22, 2022

Conversation

EvanBacon
Copy link
Contributor

Why

  • resolve ENG-6095
  • Resolve bundle identifier from app.json correctly when using npx expo start --dev-client --ios with no local ios directory

How

Test Plan

Checklist

…ing `npx expo start --dev-client --ios` with no local `ios` directory
@linear
Copy link

linear bot commented Aug 22, 2022

ENG-6095 Unable to launch to dev client build from terminal UI in local CLI

  1. Create a new dev client build on EAS Build (or create it locally and delete the ios directory)
  2. Run npx expo start --dev-client
  3. Press i in the terminal UI

image.png

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Aug 22, 2022
Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
@@ -4,6 +4,8 @@ import fs from 'fs';

import { AppIdResolver } from '../AppIdResolver';

const debug = require('debug')('expo:start:platforms:ios:AppleAppIdResolver') as typeof console.log;
Copy link
Member

Choose a reason for hiding this comment

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

is the debug type missing here, or is this caused by the require?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

caused by require instead of using import

@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Aug 22, 2022
@EvanBacon EvanBacon merged commit 60392ac into main Aug 22, 2022
@EvanBacon EvanBacon deleted the @evanbacon/cli/fix-ios-dev-client-opening-bug branch August 22, 2022 13:07
brentvatne pushed a commit that referenced this pull request Aug 22, 2022
* fix(cli): resolve bundle identifier from `app.json` correctly when using `npx expo start --dev-client --ios` with no local `ios` directory

* Update packages/@expo/cli/CHANGELOG.md

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
Ddv0623 pushed a commit to preciofishbone/expo that referenced this pull request Sep 26, 2022
* fix(cli): resolve bundle identifier from `app.json` correctly when using `npx expo start --dev-client --ios` with no local `ios` directory

* Update packages/@expo/cli/CHANGELOG.md

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants