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

[RN 0.70] Yargs as a devDependency breaks CI builds. #35893

Closed
cjhines opened this issue Jan 19, 2023 · 2 comments
Closed

[RN 0.70] Yargs as a devDependency breaks CI builds. #35893

cjhines opened this issue Jan 19, 2023 · 2 comments

Comments

@cjhines
Copy link

cjhines commented Jan 19, 2023

Description

This this merge request yargs is a dev dependency. Is it used in the generate-specs-cli script.

This will cause CI builds using npm ci to fail on this script phase.

Error: Cannot find module 'yargs'

Version

0.70.6

Output of npx react-native info

System:
OS: macOS 12.6.2
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 345.89 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.1.2 - ~/.nvm/versions/node/v16.13.2/bin/npm
Watchman: 2023.01.02.00 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /Users/admin/.rvm/gems/ruby-2.7.5/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK:
API Levels: 25, 27, 28, 29, 30, 31, 32, 33
Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.1, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-23 | Google APIs Intel x86 Atom, android-24 | Google Play Intel x86 Atom, android-27 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom, android-31 | Google APIs Intel x86 Atom_64, android-31 | Google Play Intel x86 Atom_64, android-33 | Google APIs Intel x86 Atom_64
Android NDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 11.0.17 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.6 => 0.70.6
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

Steps to reproduce

  • npm ci
  • build development build

Snack, code example, screenshot, or link to a repository

Building on CI with the following configuration will cause an error:

    build_app(
      scheme: "myapp",
      export_method: "development",
      archive_path: "./myapp.xcarchive",
      output_directory: "./development",
      include_bitcode: false,
      suppress_xcode_output: true,
      include_symbols: false,
      silent: false
    )
    ```
@lunaleaps
Copy link
Contributor

Thanks for flagging, want to submit a fix?

@cjhines
Copy link
Author

cjhines commented Jan 20, 2023

Submitted!

facebook-github-bot pushed a commit that referenced this issue Mar 2, 2023
Summary:
yargs depdendency is unavailable on npm ci breaking certain CI builds.

Fixes #35893 (comment)

Changelog: [Internal]

Prevent script failures on certain npm CI setups

Pull Request resolved: #35912

Test Plan: waitforsandcastle

Reviewed By: hoxyq

Differential Revision: D43730837

Pulled By: cortinico

fbshipit-source-id: 926182ee8482cd0c7fd183ea3e993a9322680567
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this issue May 22, 2023
Summary:
yargs depdendency is unavailable on npm ci breaking certain CI builds.

Fixes facebook#35893 (comment)

Changelog: [Internal]

Prevent script failures on certain npm CI setups

Pull Request resolved: facebook#35912

Test Plan: waitforsandcastle

Reviewed By: hoxyq

Differential Revision: D43186870

Pulled By: cortinico

fbshipit-source-id: d7a64daac71b7554780152ed2150094add62ef0d
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this issue May 22, 2023
Summary:
yargs depdendency is unavailable on npm ci breaking certain CI builds.

Fixes facebook#35893 (comment)

Changelog: [Internal]

Prevent script failures on certain npm CI setups

Pull Request resolved: facebook#35912

Test Plan: waitforsandcastle

Reviewed By: hoxyq

Differential Revision: D43730837

Pulled By: cortinico

fbshipit-source-id: 926182ee8482cd0c7fd183ea3e993a9322680567
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants