Skip to content

Commit

Permalink
include android in npm package (#30451)
Browse files Browse the repository at this point in the history
Summary:
RN app expects **android** folder with maven artifacts in NPM package, and *test-manual-e2e.sh* fails on **0.64-stable** because it couldn't find **android** folder, therefore looks into remote repos and finds 0.20.1 version on bintray.

See https://github.com/facebook/react-native/blob/e99b8bbb404f8cd1f11b6c7998083be530d7b8a4/template/android/build.gradle#L27

This PR will change npm pack to include **android** folder with maven artifacts in NPM package, thus RN android apps can find RN in node_modules.

## Changelog

[Internal] [Changed] - include *android* in npm package

Pull Request resolved: #30451

Test Plan: With this change  *test-manual-e2e.sh* will run successfully, previous it was failing due to duplicate classes error.

Reviewed By: MichaReiser

Differential Revision: D25157286

Pulled By: cpojer

fbshipit-source-id: 42fcd09d417560ebd9d3fc8fc52e1e291fca152e
  • Loading branch information
dulmandakh authored and facebook-github-bot committed Nov 24, 2020
1 parent baa8248 commit 0675bee
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
"!template/node_modules",
"!template/package-lock.json",
"!template/yarn.lock",
"android",
"cli.js",
"flow",
"flow-typed",
Expand Down

0 comments on commit 0675bee

Please sign in to comment.