Skip to content

Add --xml flag to persist xml artifacts for testing purposes (#56238)#56238

Open
coado wants to merge 7 commits intofacebook:mainfrom
coado:export-D98289956
Open

Add --xml flag to persist xml artifacts for testing purposes (#56238)#56238
coado wants to merge 7 commits intofacebook:mainfrom
coado:export-D98289956

Conversation

@coado
Copy link
Copy Markdown
Contributor

@coado coado commented Mar 27, 2026

Summary:

Currently, all xml artifacts generated by doxygen are stored in the tmp dir and deleted at the end of the snapshot generation. For debugging reasons, this diff adds --xml flag to persist generated artifacts, so that they can be analyzed.

Changelog:
[Internal]

Reviewed By: cipolleschi

Differential Revision: D98289956

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

coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
…ook#56238)

Summary:
Pull Request resolved: facebook#56238

Currently, all xml artifacts generated by doxygen are stored in the tmp dir and deleted at the end of the snapshot generation. For debugging reasons, this diff adds `--xml` flag to persist generated artifacts, so that they can be analyzed.

Changelog:
[Internal]

Differential Revision: D98289956
@coado coado force-pushed the export-D98289956 branch from 063c9d3 to 289277e Compare March 27, 2026 10:04
@meta-codesync meta-codesync bot changed the title Add --xml flag to persist xml artifacts for testing purposes Add --xml flag to persist xml artifacts for testing purposes (#56238) Mar 27, 2026
Dawid Małecki and others added 7 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
Summary:
Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Differential Revision: D97466201
Summary:
Removes stubs from the snapshot which should be irrelevant in context of the public surface.

Changelog:
[Internal]

Differential Revision: D97478132
…ook#56238)

Summary:
Pull Request resolved: facebook#56238

Currently, all xml artifacts generated by doxygen are stored in the tmp dir and deleted at the end of the snapshot generation. For debugging reasons, this diff adds `--xml` flag to persist generated artifacts, so that they can be analyzed.

Changelog:
[Internal]

Reviewed By: cipolleschi

Differential Revision: D98289956
@coado coado force-pushed the export-D98289956 branch from 289277e to aa2c503 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.

1 participant