Skip to content

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

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

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
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]

Reviewed By: cipolleschi

Differential Revision: D98289956
@coado coado force-pushed the export-D98289956 branch from 289277e to aa2c503 Compare March 27, 2026 11:13
Dawid Małecki and others added 7 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
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 aa2c503 to c0f4215 Compare March 31, 2026 11:36
@meta-codesync meta-codesync bot closed this in 1d6e68b 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 1d6e68b.

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.

1 participant