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

react-native-git-upgrade doesn't change template files, only react-upgrade command does #22538

Closed
3 tasks done
izikandrw opened this issue Dec 6, 2018 · 2 comments
Closed
3 tasks done
Labels
Resolution: Locked This issue was locked by the bot.

Comments

@izikandrw
Copy link

Environment

React Native Environment Info:
System:
OS: macOS 10.14
CPU: x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Memory: 818.50 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.7.0 - ~/.nvm/versions/node/v10.7.0/bin/node
Yarn: 1.10.1 - /usr/local/bin/yarn
npm: 6.4.1 - ~/.nvm/versions/node/v10.7.0/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
IDEs:
Android Studio: 3.2 AI-181.5540.7.32.5056338
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.5.0 => 16.5.0
react-native: 0.57.7 => 0.57.7
npmGlobalPackages:
react-native-git-upgrade: 0.2.7

Description

Summary

  • when I upgrade using react-native-git-upgrade many of the template files are not changed even though they seem like the should be. I think they should be getting upgraded because of the diffs on https://github.com/ncuillery/rn-diff and my own manual running of the react-native upgradecommand in a copy of my repo and looking at the git diff.

small example of this is in .flowconfig. When I run react-native-git-upgrade .flowconfig is not changed. When I run react-native upgrade these lines esproposal.optional_chaining=enable esproposal.nullish_coalescing=enable are added.

another example of this is in build.grade.
With react-native-git-upgrade it doesn't touch this file. With react-native upgrade many lines are added including implementation fileTree(dir: "libs", include: ["*.jar"]) implementation "com.android.support:appcompat-v7:${rootProject.ext.supportLibVersion}" implementation "com.facebook.react:react-native:+" // From node_modules

What could be the cause of these kinds of discrepancies?

@kelset kelset added the 💻CLI label Dec 18, 2018
@kelset
Copy link
Contributor

kelset commented Dec 18, 2018

👋 currently react-native-git-upgrade is a bit outdated / not maintained, so we are discussing it over here.

Over there you can find also alternative/temporary solutions.

But long term plan is to add a new version of the upgrade command to the new CLI.
Hope this can help 😇

@hramos
Copy link
Contributor

hramos commented Jan 14, 2019

The command line tools have their own repository and issue tracker over at https://github.com/react-native-community/react-native-cli.

@hramos hramos closed this as completed Jan 14, 2019
@facebook facebook locked as resolved and limited conversation to collaborators Jan 14, 2020
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Jan 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests

4 participants