Skip to content

Remove test code from the cxx-api snapshot (#56173)#56173

Closed
coado wants to merge 3 commits into
facebook:mainfrom
coado:export-D97457090
Closed

Remove test code from the cxx-api snapshot (#56173)#56173
coado wants to merge 3 commits into
facebook:mainfrom
coado:export-D97457090

Conversation

@coado
Copy link
Copy Markdown
Contributor

@coado coado commented Mar 20, 2026

Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090

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

meta-codesync Bot commented Mar 20, 2026

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

@facebook-github-tools facebook-github-tools Bot added the p: Facebook Partner: Facebook label Mar 20, 2026
@meta-codesync meta-codesync Bot changed the title Remove test code from the cxx-api snapshot Remove test code from the cxx-api snapshot (#56173) Mar 20, 2026
coado added a commit to coado/react-native that referenced this pull request Mar 20, 2026
Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from f742e53 to 1691d83 Compare March 20, 2026 11:34
coado added a commit to coado/react-native that referenced this pull request Mar 20, 2026
Summary:
Pull Request resolved: facebook#56173

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from 1691d83 to d90979f Compare March 20, 2026 11:38
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 25, 2026
Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from d90979f to beb6c69 Compare March 25, 2026 15:39
coado added a commit to coado/react-native that referenced this pull request Mar 25, 2026
Summary:
Pull Request resolved: facebook#56173

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from beb6c69 to 6aaa25b Compare March 25, 2026 15:43
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from 6aaa25b to a0f6e67 Compare March 27, 2026 09:10
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:
Pull Request resolved: facebook#56173

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from a0f6e67 to 7c11617 Compare March 27, 2026 09:14
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from 7c11617 to b704d6b 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#56173

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from b704d6b to 8f949e1 Compare March 27, 2026 11:13
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
@coado coado force-pushed the export-D97457090 branch from 8f949e1 to 8c330a9 Compare March 31, 2026 11:32
coado added a commit to coado/react-native that referenced this pull request Mar 31, 2026
Summary:

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

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

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
Summary:
Pull Request resolved: facebook#56173

Removes leaked test code from the snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97457090
@coado coado force-pushed the export-D97457090 branch from 8c330a9 to 2b905eb Compare March 31, 2026 11:38
@meta-codesync meta-codesync Bot closed this in 3a80179 Mar 31, 2026
@facebook-github-tools facebook-github-tools Bot added the Merged This PR has been merged. label Mar 31, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 31, 2026

This pull request has been merged in 3a80179.

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