question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Consider renaming the app_name to match the RN template and simplify the diff

See original GitHub issue

Feature Request

Currently the diff is made between an RN app named RnDiffApp and the RN template which has a different app_name/DisplayName of Hello App Display Name. When upgrading this shows as 2 additional diffs that are IMO not necessary, see the SS:

rn_upgrade_helper_app_name_diff

I propose we handle this name discrepancy so that it doesn’t show up in the diff.

Why it is needed

Every upgrade shows at least 2 diffs that will (always?) be ignored. This unnecessarily increases the cognitive load for each upgrade.

Possible implementation

I thought the change would be as simple as running react-native init „Hello App Display Name” but the projectName does not allow spaces. One solution that comes to mind is replacing RnDiffApp (which is a fairly specific string) with Hello App Display Name. However I’m not sure if we want to add this to the workflow.

Another option would be to submit a patch directly to https://github.com/facebook/react-native/tree/master/template and change the name to something like HelloWorld. This doesn’t feel right, though 😃

Let me know if this makes sense and if you have better options on how to handle this.

Issue Analytics

  • State:closed
  • Created 4 years ago
  • Comments:6 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
lucasbentocommented, Jul 23, 2019

@rszalski: we added a small comment on the files explaining that this change must be ignored, it’s currently being tracked at react-native-community/cli#561 as mentioned before and as soon as it’s fixed on the CLI we will update the diff and remove the note.

Again, thank you for opening this issue.

1reaction
lucasbentocommented, Jul 23, 2019

This is related to react-native-community/cli#561.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Change App Name In React Native - Stack Overflow
I find that, if you want to change both the app name and the package name (i.e. rename the entire app), the following...
Read more >
Improvement: Adjust template to match new init command by ...
Summary Adjusting template to new init command (react-native-community/cli#241). PR with new init command needs to be merged before we land this.
Read more >
Difference Quotient - How to Simplify (3 Types) - YouTube
Learn how to simplify the difference quotient in this video math tutorial by Mario's Math Tutoring. We go through 3 examples : one...
Read more >
FTPortal on the Mac App Store
FTPortal expedites uploading files to specific directories of an FTP server by creating and configuring multiple copies of the app. Then you ...
Read more >
Health Cloud Administration - Salesforce Implementation guides
We've built the Health Cloud console to be a powerful tool that you can tailor to fit how your organization manages people and...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found