react-native/local-cli/link/android
Thibault Malbranche dbd47592a1 Local cli/android/normalize project name
Summary:
<!--
Thank you for sending the PR! We appreciate you spending the time to work on these changes.

Help us understand your motivation by explaining why you decided to make this change.

You can learn more about contributing to React Native here: http://facebook.github.io/react-native/docs/contributing.html

Happy contributing!

-->

Scoped packages are starting to be the new thing, and gradle does not work properly with '/' in the project name, so this PR links them and replaces '/' by '_' . This only affects android.

I added tests in the 2 impacted functions + a test file for the normalizer function

<!--
Help reviewers and the release process by writing your own release notes

**INTERNAL and MINOR tagged notes will not be included in the next version's final release notes.**

-->
[CLI] [BUGFIX] [local-cli/link/link.js] - On android, Scoped packages will now get the '/' replaced with '_' to ensure gradle works nicely.  ⚠️ However if you previously linked scoped packages, they will get linked again. ⚠️
Closes https://github.com/facebook/react-native/pull/18275

Differential Revision: D7305227

Pulled By: hramos

fbshipit-source-id: 1c95563e884175529692948b29407a7733c44353
2018-03-16 11:26:01 -07:00
..
patches Local cli/android/normalize project name 2018-03-16 11:26:01 -07:00
copyAssets.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00
fs.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00
index.js Uses a single code path to link and unlink all platforms 2018-02-13 05:06:42 -08:00
isInstalled.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00
registerNativeModule.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00
unlinkAssets.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00
unregisterNativeModule.js Update license headers for MIT license 2018-02-16 18:31:53 -08:00