Skip to content

Exclude metainternal from C++ API snapshots (#56545)#56545

Closed
lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D101879633
Closed

Exclude metainternal from C++ API snapshots (#56545)#56545
lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D101879633

Conversation

@lavenzg

@lavenzg lavenzg commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Summary:

Add /metainternal/ to top-level exclude_patterns in the C++ API snapshot config.
This directory may contain internal headers that are not part of the public React Native
C++ API surface.

Changelog: [Internal]

Differential Revision: D101879633

@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 Apr 21, 2026
@facebook-github-tools facebook-github-tools Bot added p: Facebook Partner: Facebook Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Apr 21, 2026
@lavenzg lavenzg force-pushed the export-D101879633 branch 2 times, most recently from e33e5a1 to db96246 Compare April 21, 2026 23:22
@lavenzg lavenzg force-pushed the export-D101879633 branch from db96246 to df12510 Compare April 21, 2026 23:27
@meta-codesync

meta-codesync Bot commented Apr 21, 2026

Copy link
Copy Markdown

@lavenzg has exported this pull request. If you are a Meta employee, you can view the originating Diff in D101879633.

lavenzg added a commit to lavenzg/react-native that referenced this pull request Apr 21, 2026
Summary:
Pull Request resolved: react#56545

Add */metainternal/* to ReactApple's exclude_patterns in the C++ API snapshot config.
This directory may contain internal headers that are not part of the public React Native
C++ API surface.

Changelog: [Internal]

Differential Revision: D101879633
@meta-codesync meta-codesync Bot changed the title Exclude metainternal from C++ API snapshots Exclude metainternal from C++ API snapshots (#56545) Apr 21, 2026
@lavenzg lavenzg force-pushed the export-D101879633 branch from df12510 to ed30780 Compare April 21, 2026 23:33
Summary:

Add */metainternal/* to top-level exclude_patterns in the C++ API snapshot config. 
This directory may contain internal headers that are not part of the public React Native
C++ API surface.

Changelog: [Internal]

Differential Revision: D101879633
@lavenzg lavenzg force-pushed the export-D101879633 branch from ed30780 to f8336f7 Compare April 22, 2026 16:04

@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 a78d6f4 Apr 22, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @lavenzg in a78d6f4

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 22, 2026
@meta-codesync

meta-codesync Bot commented Apr 22, 2026

Copy link
Copy Markdown

This pull request has been merged in a78d6f4.

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. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants