Skip to content

Remove fantom test specific methods from the C++ API snapshot (#56247)#56247

Open
coado wants to merge 5 commits intofacebook:mainfrom
coado:export-D97466201
Open

Remove fantom test specific methods from the C++ API snapshot (#56247)#56247
coado wants to merge 5 commits intofacebook:mainfrom
coado:export-D97466201

Conversation

@coado
Copy link
Copy Markdown
Contributor

@coado coado commented Mar 27, 2026

Summary:

Removes fantom test specific methods, namely:

  • NativeFantomTestSpecificMethods,
  • FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201

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

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.

Dawid Małecki added 4 commits March 27, 2026 03:51
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
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]

Differential Revision: D97465431
@meta-codesync meta-codesync bot changed the title Remove fantom test specific methods from the C++ API snapshot Remove fantom test specific methods from the C++ API snapshot (#56247) Mar 27, 2026
@coado coado force-pushed the export-D97466201 branch from e7e6827 to b8108fa Compare March 27, 2026 11:09
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
…ok#56247)

Summary:

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
…ok#56247)

Summary:
Pull Request resolved: facebook#56247

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
@coado coado force-pushed the export-D97466201 branch from b8108fa to eb9c7ce Compare March 27, 2026 11:13
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 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