Skip to content

Cover react/renderer/consistency:consistency with Stable API guards - #58046

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

Cover react/renderer/consistency:consistency with Stable API guards#58046
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D116601683

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary:
Classifies react/renderer/consistency:consistency as a private target under the C++ stable API three-tier visibility model. Adds #include <react/cxxstableapi/PrivateGuard.h> to both headers in the module and declares the guard dependency in BUCK, CMake and the podspec.

Changelog: [Internal]

Differential Revision: D116601683

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

meta-codesync Bot commented Aug 21, 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 D116601683.

@facebook-github-tools facebook-github-tools Bot added the p: Facebook Partner: Facebook label Aug 21, 2026
Summary:
Classifies `react/renderer/consistency:consistency` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to both headers in the module and declares the guard dependency in BUCK, CMake and the podspec.

Changelog: [Internal]

Differential Revision: D116601683

@cortinico cortinico 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.

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in bc64b26 Sep 8, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 8, 2026
@meta-codesync

meta-codesync Bot commented Sep 8, 2026

Copy link
Copy Markdown

This pull request has been merged in bc64b26.

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. Merged This PR has been merged. 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