Skip to content

Cherry-pick Yoga UDL fix for beta-0.1.1#111

Open
clholgat wants to merge 1 commit into
release/beta-0.1from
cherry-pick/yoga-udl-fix
Open

Cherry-pick Yoga UDL fix for beta-0.1.1#111
clholgat wants to merge 1 commit into
release/beta-0.1from
cherry-pick/yoga-udl-fix

Conversation

@clholgat

Copy link
Copy Markdown
Collaborator

Summary

Cherry-pick the Yoga YGValue.h UDL syntax fix from main (1c6d7408) to the release branch.

Newer Clang versions (shipped with recent macOS GitHub Actions runners) treat operator"" _pt (with space before the suffix) as -Werror,-Wdeprecated-literal-operator. This removes the space so it compiles cleanly.

This was breaking:

  • Valdi CI smoke tests (iOS build)
  • Publish NPM release test (iOS build)
  • Release Test workflow

Test Plan

CI on this PR should pass. After merge, beta-0.1.1 will be retagged at the new branch head.

Newer Clang versions treat `operator"" _pt` (with space) as
-Werror,-Wdeprecated-literal-operator. Remove the space so it
compiles cleanly: `operator""_pt`.

Cherry-picked from main (1c6d740).
@github-actions

Copy link
Copy Markdown

Sensitive Files Detected

🧘 Yoga layout engine — HIGH blast radius — affects all layout. Verify layout tests exist and pass.

This is an automated notice. A maintainer will review after import.

@github-actions github-actions Bot added area/yoga Yoga layout engine size/S labels Jun 29, 2026
@github-actions

Copy link
Copy Markdown

📊 PR Size: size/S

Total changes: 12 lines (1 files)

Top files changed:

  • third-party/yoga/src/yoga/YGValue.h: +6 -6

Size calculated as additions + deletions. Labels: XS (<10), S (<50), M (<250), L (<1000), XL (1000+)

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

⚠️ Bazel & CI Test Results

Test Suite Result
Valdi Smoke Tests ❌ failure
Snapshot Tests ✅ success
Linux: Build & Export ✅ success
Linux: C++ Tests ✅ success

Some tests failed. Please check the workflow logs for details.

🚀 Bazel remote cache is now enabled - future builds will be faster!

Workflow: Valdi CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/yoga Yoga layout engine size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant