Skip to content

Conversation

@microsoft-github-policy-service microsoft-github-policy-service bot added Area: Fabric Support Facebook Fabric New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric labels Aug 15, 2024
@TatianaKapos TatianaKapos changed the title [Draft] Integrate 7/15 Integrate RN Nightly 7/19 Aug 16, 2024
@TatianaKapos TatianaKapos marked this pull request as ready for review August 16, 2024 23:27
@TatianaKapos TatianaKapos requested review from a team as code owners August 16, 2024 23:27
"application/javascript" // Not in Chromium but emitted by Metro
};

// namespace { [Windows #13587]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I was reading about anonymous-namespaces, this file should be able to scope to these variables but VS throws anonymous-namespace errors. Is there a better fix for this?
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not blocking the integration but will look more into this later :)

@TatianaKapos TatianaKapos enabled auto-merge (squash) August 21, 2024 23:13
@TatianaKapos TatianaKapos merged commit bccbec0 into microsoft:main Aug 21, 2024
@TatianaKapos TatianaKapos deleted the tk-integrate-7-12 branch August 21, 2024 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Fabric Support Facebook Fabric New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Unfork Yoga layout files

4 participants