Skip to content

Use module umbrella headers in attributedstring (#58814) - #58814

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

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

Conversation

@j-piasecki

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

Copy link
Copy Markdown
Contributor

Summary:

The attributedstring headers included fine-grained headers from other modules that are protected by UmbrellaGuard.h, which hard-errors for consumers building with RN_STRICT_API. Include the module umbrella headers instead.

Changelog: [Internal]

Differential Revision: D123001117

@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 2, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Oct 2, 2026
@meta-codesync

meta-codesync Bot commented Oct 2, 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 D123001117.

poriyavaibhav2611

This comment was marked as low quality.

Summary:
Pull Request resolved: react#58814

The `attributedstring` headers included fine-grained headers from other modules that are protected by `UmbrellaGuard.h`, which hard-errors for consumers building with `RN_STRICT_API`. Include the module umbrella headers instead.

Changelog: [Internal]

Differential Revision: D123001117
@meta-codesync meta-codesync Bot changed the title Use module umbrella headers in attributedstring Use module umbrella headers in attributedstring (#58814) Oct 2, 2026

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.

2 participants