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

Upgrade React Native to 0.63 #14776

Closed
wants to merge 35 commits into from
Closed

Upgrade React Native to 0.63 #14776

wants to merge 35 commits into from

Conversation

ceyhun
Copy link
Contributor

@ceyhun ceyhun commented Aug 27, 2020

Upgrading react-native to version 0.63.2

To test: Follow instructions on gutenberg-mobile PR: wordpress-mobile/gutenberg-mobile#2580

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

marecar3 and others added 28 commits March 26, 2020 22:12
# Conflicts:
#	Podfile
#	Podfile.lock
# Conflicts:
#	Podfile
#	Podfile.lock
# Conflicts:
#	Gemfile.lock
#	Podfile
#	Podfile.lock
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 27, 2020

You can trigger an installable build for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 27, 2020

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS

@ceyhun ceyhun added the Gutenberg Editing and display of Gutenberg blocks. label Aug 27, 2020
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 27, 2020

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@marecar3 marecar3 mentioned this pull request Oct 12, 2020
3 tasks
%x(patch "#{project_root}/Pods/React-CoreModules/RCTActionSheetManager.mm" < "#{project_root}/patches/RN-RCTActionSheetManager.patch")
puts 'Patching FBReactNativeSpec to add possibility to disable action sheet buttons -
it could be removed once PR with that functionality will be merged into RN'
%x(patch "#{project_root}/Pods/FBReactNativeSpec/FBReactNativeSpec/FBReactNativeSpec.h" < "#{project_root}/patches/FBReactNativeSpec.patch")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might fall too far outside of the scope of what we're trying to take on here (update to RN 0.63) but I wonder if it would be possible to apply the suggested changes from #14240 since we're modifying these patches.

#14240

So, instead of using %x(...) can we experiment with replacing that line with system('patch ... < ...') || exit(1). system(...) returns the exit value of the shell so it can help us detecting when the patch fails. But, we should make sure it is not producing false positives and causing the termination of the process unnecessarily.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think with the 0.64 upgrade none of these patches here will be necessary anymore. This final remaining patch seems to be merged into 0.64.

@ceyhun ceyhun closed this Jul 22, 2021
@ceyhun ceyhun deleted the gutenberg/upgrade-RN-0-63 branch July 22, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gutenberg Editing and display of Gutenberg blocks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants