Skip to content

Retier react/renderer/uimanager headers (#58846) - #58846

Open
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D123391983
Open

j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D123391983

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Make AppRegistryBinding.h, LayoutEventEmitter.h and the deprecated SurfaceRegistryBinding.h private; they are only used inside React Native. UIManagerViewTransitionDelegate.h now reaches Float.h through the <React/Graphics.h> umbrella instead of including it directly, since Float.h is public and its guard rejects direct includes from for-frameworks headers.

Changelog: [Internal]

Differential Revision: D123391983

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 5, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner labels Oct 5, 2026
@meta-codesync

meta-codesync Bot commented Oct 5, 2026

Copy link
Copy Markdown

@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D123391983.

@facebook-github-tools facebook-github-tools Bot added the p: Facebook Partner: Facebook label Oct 5, 2026
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Oct 5, 2026
Summary:
Pull Request resolved: react#58846

Make `AppRegistryBinding.h`, `LayoutEventEmitter.h` and the deprecated `SurfaceRegistryBinding.h` private; they are only used inside React Native. `UIManagerViewTransitionDelegate.h` now reaches `Float.h` through the `<React/Graphics.h>` umbrella instead of including it directly, since `Float.h` is public and its guard rejects direct includes from for-frameworks headers.

Changelog: [Internal]

Differential Revision: D123391983
@meta-codesync meta-codesync Bot changed the title Retier react/renderer/uimanager headers Retier react/renderer/uimanager headers (#58846) Oct 5, 2026
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Oct 5, 2026
Summary:

Make `AppRegistryBinding.h`, `LayoutEventEmitter.h` and the deprecated `SurfaceRegistryBinding.h` private; they are only used inside React Native. `UIManagerViewTransitionDelegate.h` now reaches `Float.h` through the `<React/Graphics.h>` umbrella instead of including it directly, since `Float.h` is public and its guard rejects direct includes from for-frameworks headers.

Changelog: [Internal]

Differential Revision: D123391983
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Oct 5, 2026
Summary:

Make `AppRegistryBinding.h`, `LayoutEventEmitter.h` and the deprecated `SurfaceRegistryBinding.h` private; they are only used inside React Native. `UIManagerViewTransitionDelegate.h` now reaches `Float.h` through the `<React/Graphics.h>` umbrella instead of including it directly, since `Float.h` is public and its guard rejects direct includes from for-frameworks headers.

Changelog: [Internal]

Differential Revision: D123391983
Summary:

Make `AppRegistryBinding.h`, `LayoutEventEmitter.h` and the deprecated `SurfaceRegistryBinding.h` private; they are only used inside React Native. `UIManagerViewTransitionDelegate.h` now reaches `Float.h` through the `<React/Graphics.h>` umbrella instead of including it directly, since `Float.h` is public and its guard rejects direct includes from for-frameworks headers.

Changelog: [Internal]

Differential Revision: D123391983

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant