Skip to content

[Gutenberg] Integrate React Native 0.66 upgrade of Gutenberg Mobile#17594

Merged
dcalhoun merged 9 commits intodevelopfrom
chore/upgrade-to-react-native-0.66
Jan 3, 2022
Merged

[Gutenberg] Integrate React Native 0.66 upgrade of Gutenberg Mobile#17594
dcalhoun merged 9 commits intodevelopfrom
chore/upgrade-to-react-native-0.66

Conversation

@twstokes
Copy link
Copy Markdown
Contributor

@twstokes twstokes commented Nov 29, 2021

Update Gutenberg Mobile to include the upgrade to React Native 0.66.

Related PRs

Regression Notes

  1. Potential unintended areas of impact
    Regressions in the block editor from upgrading the foundational third-party dependency utilized.
  2. What I did to test those areas of impact (or what existing automated tests I relied on)
    Performed Gutenberg Mobile Writing Flow, UBE, and Sanity test cases.
  3. What automated tests I added (or what prevented me from doing so)
    Not really possible or relevant for this type of dependency upgrade work.

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • 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.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Nov 29, 2021

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

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Nov 29, 2021

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

Generated by 🚫 dangerJS

@twstokes twstokes force-pushed the chore/upgrade-to-react-native-0.66 branch from c75baef to cf1653b Compare November 29, 2021 22:10
@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Nov 29, 2021

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

@twstokes twstokes force-pushed the chore/upgrade-to-react-native-0.66 branch from cf1653b to 7cc8462 Compare November 30, 2021 14:10
@@ -472,9 +474,12 @@ post_install do |installer|
# =====================================
#
installer.pods_project.targets.each do |target|
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't work to add this tweak because I believe this project now only compiles for Xcode 13+.

@twstokes twstokes force-pushed the chore/upgrade-to-react-native-0.66 branch from 7cc8462 to 2f21583 Compare November 30, 2021 22:49
@twstokes twstokes self-assigned this Nov 30, 2021
@twstokes twstokes force-pushed the chore/upgrade-to-react-native-0.66 branch 2 times, most recently from 6601b68 to caa776d Compare November 30, 2021 23:41
@twstokes twstokes force-pushed the chore/upgrade-to-react-native-0.66 branch from caa776d to 0a66cf2 Compare December 1, 2021 16:13
Updates Pods to include the latest generated Podspec files for
`gutenberg-mobile`.
@twstokes twstokes changed the title [WIP RN] Add dependencies for RN 0.66. [Gutenberg] Integrate React Native 0.66 upgrade of Gutenberg Mobile Dec 1, 2021
@dcalhoun dcalhoun marked this pull request as ready for review December 6, 2021 23:14
@dcalhoun dcalhoun requested review from dcalhoun and hypest December 9, 2021 15:59
Copy link
Copy Markdown
Member

@dcalhoun dcalhoun left a comment

Choose a reason for hiding this comment

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

The changes in this PR make sense to me. Much of the WPiOS Writing Flow, Unsupported Block Editor, and additional features were tested against wordpress-mobile/gutenberg-mobile#4235.

LGTM. Before we merge this PR, we need to update the gutenberg Podfile reference to a new tag.

@dcalhoun dcalhoun assigned dcalhoun and unassigned twstokes Jan 3, 2022
@dcalhoun dcalhoun enabled auto-merge January 3, 2022 23:20
@dcalhoun dcalhoun merged commit 1657e03 into develop Jan 3, 2022
@dcalhoun dcalhoun deleted the chore/upgrade-to-react-native-0.66 branch January 3, 2022 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants