Skip to content

Integrate June 9 Nightly RN Build#8147

Merged
10 commits merged into
microsoft:mainfrom
NickGerleman:integrate-6-9
Jul 15, 2021
Merged

Integrate June 9 Nightly RN Build#8147
10 commits merged into
microsoft:mainfrom
NickGerleman:integrate-6-9

Conversation

@NickGerleman
Copy link
Copy Markdown
Contributor

@NickGerleman NickGerleman commented Jul 1, 2021

Commits: facebook/react-native@0d32aef...386dbd9

Has ACoates fixes for RNTester theming, and testID in NewAppScreen header.

Will manuallly validate that the new RNTester example in facebook/react-native@7463f6d works as expected on UWP. It does not run on NetUI due to using instance-global dimension APIs.

Microsoft Reviewers: Open in CodeFlow

Commits: facebook/react-native@0d32aef...386dbd9

Has ACoates fixes for RNTester theming, and testID in NewAppScreen header.

Will manuallly validate that the new RNTester example in facebook/react-native@7463f6d works as expected on UWP. It does not run on NetUI due to using instance-global dimension APIs.
@NickGerleman NickGerleman requested a review from a team as a code owner July 1, 2021 17:23
@NickGerleman
Copy link
Copy Markdown
Contributor Author

Seeing fabric compile errors because someone added "#import" to C++ code again. Must be some Objective C devs leaking over 😛.

Need to fork to #include and fix upstream...

@NickGerleman
Copy link
Copy Markdown
Contributor Author

Snapshots also failing bc padding changed. Need to update those.

@NickGerleman
Copy link
Copy Markdown
Contributor Author

Upstream PR to replace #import with #include: facebook/react-native#31805

Forked in the meantime

@NickGerleman
Copy link
Copy Markdown
Contributor Author

Babel transformation of one of our e2e-test-app source files now fails for some reason. Going to try reverting babel-preset-fbjs to 3.3.0

@NickGerleman NickGerleman added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Jul 15, 2021
@ghost
Copy link
Copy Markdown

ghost commented Jul 15, 2021

Hello @NickGerleman!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit c0db5f7 into microsoft:main Jul 15, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants