Skip to content

Add global exclude patterns to the cxx api config (#56248)#56248

Closed
coado wants to merge 4 commits intofacebook:mainfrom
coado:export-D97465431
Closed

Add global exclude patterns to the cxx api config (#56248)#56248
coado wants to merge 4 commits intofacebook:mainfrom
coado:export-D97465431

Conversation

@coado
Copy link
Copy Markdown
Contributor

@coado coado commented Mar 27, 2026

Summary:

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431

@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 Mar 27, 2026
@facebook-github-tools facebook-github-tools bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Mar 27, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Mar 27, 2026

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

Copy link
Copy Markdown
Contributor

@cortinico cortinico left a comment

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.

coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
@meta-codesync meta-codesync bot changed the title Add global exclude patterns to the cxx api config Add global exclude patterns to the cxx api config (#56248) Mar 27, 2026
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
@coado coado force-pushed the export-D97465431 branch from ea4332e to 6ed11c5 Compare March 27, 2026 11:10
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:
Pull Request resolved: facebook#56248

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
@coado coado force-pushed the export-D97465431 branch from 6ed11c5 to 7b4433a Compare March 27, 2026 11:13
Dawid Małecki added 3 commits March 31, 2026 04:30
Summary:
This diff introduces initial snapshots of the Cxx public API in a form of a couple .api files, namely:

- ReactAndroidDebugCxx.api
- ReactAndroidReleaseCxx.api
- ReactAppleDebugCxx.api
- ReactAppleReleaseCxx.api
- ReactCommonDebugCxx.api
- ReactCommonReleaseCxx.api

that is for `ReactAndroid`, `ReactApple`, and shared `ReactCommon` in two variants `Release` and `Debug` each.

Differential Revision: D97296573
Summary:
Pull Request resolved: facebook#56042

Adds CI workflow for validating whether the current C++ API snapshot is equivalent with the generated one.

Changelog:
[Internal]

Differential Revision: D95963515
Summary:
Removes leaked test code from the snapshot.

Changelog:
[Internal]

Differential Revision: D97457090
coado added a commit to coado/react-native that referenced this pull request Mar 31, 2026
Summary:

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
@coado coado force-pushed the export-D97465431 branch from 7b4433a to 277f285 Compare March 31, 2026 11:33
coado added a commit to coado/react-native that referenced this pull request Mar 31, 2026
Summary:

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
Summary:
Pull Request resolved: facebook#56248

Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97465431
@coado coado force-pushed the export-D97465431 branch from 277f285 to dab79da Compare March 31, 2026 11:36
@meta-codesync meta-codesync bot closed this in b0b30b5 Mar 31, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Mar 31, 2026

This pull request has been merged in b0b30b5.

@facebook-github-tools facebook-github-tools bot added the Merged This PR has been merged. label Mar 31, 2026
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 p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants