Skip to content

Exclude Yoga tests from iOS build - #58439

Closed
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D119478374
Closed

Exclude Yoga tests from iOS build#58439
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D119478374

Conversation

@cipolleschi

@cipolleschi cipolleschi commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary:
Fixes OSS CI breakage by excluding the test directory from the Yoga SwiftPM target. The new test/BaselineTest.cpp requires gtest which is unavailable in the iOS build environment, causing compilation failures. CocoaPods and CMake builds were already unaffected.

Changelog:

[Internal] -


Agent Home session: https://www.internalfb.com/agent-home?session_id=dmh_fileadiff_diffcomment_D119293889_iDlbpMcSAiXb

Differential Revision: D119478374

Summary:
Fixes OSS CI breakage by excluding the `test` directory from the Yoga SwiftPM target. The new `test/BaselineTest.cpp` requires gtest which is unavailable in the iOS build environment, causing compilation failures. CocoaPods and CMake builds were already unaffected.

---
Agent Home session: https://www.internalfb.com/agent-home?session_id=dmh_fileadiff_diffcomment_D119293889_iDlbpMcSAiXb

Differential Revision: D119478374
@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 Sep 10, 2026
@meta-codesync

meta-codesync Bot commented Sep 10, 2026

Copy link
Copy Markdown

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

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

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.

@meta-codesync meta-codesync Bot closed this in 506165d Sep 10, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 10, 2026
@meta-codesync

meta-codesync Bot commented Sep 10, 2026

Copy link
Copy Markdown

This pull request has been merged in 506165d.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants