Skip to content

Commit

Permalink
Unblock yarn android on main
Browse files Browse the repository at this point in the history
Summary:
`yarn android` on main is currently broken due to the CLI attempting to search for the manifest inside the src/androidTest folder.
This fixes it by specifying the exact path of the Android Manifest.
The fix to the CLI is also pending to prevent the CLI from searching inside test folders.

Fix for the CLI is here:
- react-native-community/cli#2075

Changelog:
[Internal] [Changed] - Unblock `yarn android` on main

Differential Revision: D49190626

fbshipit-source-id: 26fcdb13147f992451d428a9b8dd886c0cb5bfd5
  • Loading branch information
cortinico authored and facebook-github-bot committed Sep 12, 2023
1 parent 5bd9486 commit 1038a1a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion packages/rn-tester/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
},
"scripts": {
"start": "react-native start",
"android": "react-native run-android --mode HermesDebug --appId 'com.facebook.react.uiapp' --main-activity 'com.facebook.react.uiapp.RNTesterActivity'",
"android": "react-native run-android --mode HermesDebug",
"install-android-jsc": "../../gradlew :packages:rn-tester:android:app:installJscDebug",
"install-android-hermes": "../../gradlew :packages:rn-tester:android:app:installHermesDebug",
"clean-android": "rm -rf android/app/build",
Expand Down
4 changes: 4 additions & 0 deletions packages/rn-tester/react-native.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,10 @@ module.exports = {
},
android: {
sourceDir: '../../',
// To remove once the CLI fix for manifestPath search path is landed.
manifestPath:
'packages/rn-tester/android/app/src/main/AndroidManifest.xml',
packageName: 'com.facebook.react.uiapp',
},
},
};

0 comments on commit 1038a1a

Please sign in to comment.