Skip to content

[iOS] Remove unused old-arch import to fix build with React Native 0.87-nightly#4247

Merged
j-piasecki merged 1 commit into
mainfrom
@jpiasecki/remove-unused-import
Jun 9, 2026
Merged

[iOS] Remove unused old-arch import to fix build with React Native 0.87-nightly#4247
j-piasecki merged 1 commit into
mainfrom
@jpiasecki/remove-unused-import

Conversation

@j-piasecki

Copy link
Copy Markdown
Member

Description

React Native 0.87 will be the first version with parts of the old architecture actually removed. The current nightly releases are failing to build with RNGH due to a stale import of an old-arch header.

Test plan

Create a new RN app using the nightly release and add RNGH.

Copilot AI review requested due to automatic review settings June 9, 2026 06:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes iOS builds against React Native 0.87 nightly by removing a stale old-architecture header import that is no longer present in newer RN versions.

Changes:

  • Remove the unused RCTRootContentView import from RNGestureHandlerManager.mm to avoid build failures on RN 0.87 nightly / upcoming 0.87.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@j-piasecki
j-piasecki merged commit 6ce4668 into main Jun 9, 2026
4 checks passed
@j-piasecki
j-piasecki deleted the @jpiasecki/remove-unused-import branch June 9, 2026 07:51
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.

3 participants