Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: bump react-native to 0.74 #3290

Merged
merged 6 commits into from
Aug 13, 2024
Merged

chore: bump react-native to 0.74 #3290

merged 6 commits into from
Aug 13, 2024

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Aug 13, 2024

Description

Bump React Native to 0.74.

react-native-macos was also removed so that we can bump to 0.75 next week.

Test plan

CI should pass.

@github-actions github-actions bot added feature: metro This is related to Metro chore Improvements that don't directly affect features feature: cli This is related to CLI feature: jest This is related to Jest feature: sdk This is related to 1RN SDK feature: webapis This is related to the experimental effort around WebAPIs for RN labels Aug 13, 2024
@github-actions github-actions bot added the feature: align-deps This is related to align-deps label Aug 13, 2024
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Aug 13, 2024
Summary:
Android fails to build in a pnpm monorepo setup because `yargs` cannot be found:

```
% yarn android
info Installing the app...

> Configure project :app
WARNING: The option setting 'android.jetifier.ignorelist=hermes-android' is experimental.
Signing config for 'release' build type not found; reusing debug config

> Task :react-native-webapis_web-storage:generateCodegenSchemaFromJavaScript FAILED
28 actionable tasks: 6 executed, 22 up-to-date

node:internal/modules/cjs/loader:1148
  throw err;
  ^

Error: Cannot find module 'yargs'
Require stack:
- /~/node_modules/.store/react-native-codegen-virtual-39ff8dcc54/package/lib/cli/combine/combine-js-to-schema-cli.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1145:15)
    at Module._load (node:internal/modules/cjs/loader:986:27)
    at Module.require (node:internal/modules/cjs/loader:1233:19)
    at require (node:internal/modules/helpers:179:18)
```

## Changelog:

[GENERAL] [FIXED] - Fix codegen failing in a pnpm monorepo because of missing `yargs`

Pull Request resolved: #45994

Test Plan: Tested in microsoft/rnx-kit#3290

Reviewed By: dmytrorykun

Differential Revision: D61201420

Pulled By: cortinico

fbshipit-source-id: aac3704ae7f200db827b14c8362f83a5e66ad08e
@tido64 tido64 merged commit d21adb8 into main Aug 13, 2024
10 checks passed
@tido64 tido64 deleted the tido/bump-0.74 branch August 13, 2024 14:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
chore Improvements that don't directly affect features feature: align-deps This is related to align-deps feature: cli This is related to CLI feature: jest This is related to Jest feature: metro This is related to Metro feature: sdk This is related to 1RN SDK feature: webapis This is related to the experimental effort around WebAPIs for RN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants