Skip to content

Revert "bump node.js version for CI, fixing useSyncExternalStoreShared-test"#23239

Merged
gaearon merged 1 commit into
mainfrom
revert-23236-main
Feb 7, 2022
Merged

Revert "bump node.js version for CI, fixing useSyncExternalStoreShared-test"#23239
gaearon merged 1 commit into
mainfrom
revert-23236-main

Conversation

@gaearon

@gaearon gaearon commented Feb 7, 2022

Copy link
Copy Markdown
Collaborator

Reverts #23236

This broke CI on main.

@gaearon
gaearon merged commit cd4eb11 into main Feb 7, 2022
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Feb 7, 2022
@sunderls

sunderls commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

oh.. sorry for this..

@gaearon

gaearon commented Feb 7, 2022

Copy link
Copy Markdown
Collaborator Author

No problem, just need to figure out why. Wanna look into the failure?

@sunderls

sunderls commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

yeah, will try to figure it out
I think I got some clues, will PR again soon.

@sunderls

sunderls commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

tried again here #23243
hope this would work, I checked yarn test --prod this time.

facebook-github-bot pushed a commit to react/react-native that referenced this pull request Feb 9, 2022
Summary:
This sync includes the following changes:
- **[a3bde79](react/react@a3bde7974 )**: Exclude react-dom/unstable_testing entry point from stable releases ([#23258](react/react#23258)) //<Sebastian Markbåge>//
- **[5690932](react/react@569093276 )**: Add onErrorShell Callback ([#23247](react/react#23247)) //<Sebastian Markbåge>//
- **[0dedfcc](react/react@0dedfcc68 )**: Update the exports field ([#23257](react/react#23257)) //<Sebastian Markbåge>//
- **[9d4e8e8](react/react@9d4e8e84f )**: React Native raw event EventEmitter - intended for app-specific perf listeners and debugging ([#23232](react/react#23232)) //<Joshua Gross>//
- **[1dece52](react/react@1dece5235 )**: Add back warning with component stack on Hydration mismatch ([#23241](react/react#23241)) //<salazarm>//
- **[cd4eb11](react/react@cd4eb116c )**: Revert "update node.js version for CI ([#23236](react/react#23236))" ([#23239](react/react#23239)) //<dan>//
- **[1d7728b](react/react@1d7728bf9 )**: update node.js version for CI ([#23236](react/react#23236)) //<sunderls>//
- **[848e802](react/react@848e802d2 )**: Add onRecoverableError option to hydrateRoot, createRoot ([#23207](react/react#23207)) //<Andrew Clark>//
- **[5318971](react/react@5318971f5 )**: Remove logic for multiple error recovery attempts ([#23227](react/react#23227)) //<Andrew Clark>//
- **[3a44621](react/react@3a4462129 )**: Disable avoidThisFallback support in Fizz  ([#23224](react/react#23224)) //<salazarm>//
- **[0318ac2](react/react@0318ac2c4 )**: Revert 4f5449 //<Ricky>//
- **[4f5449e](react/react@4f5449eb4 )**: Remove main from scheduler `index.js` //<Ricky>//
- **[3f5ff16](react/react@3f5ff16c1 )**: [Hydration] Fallback to client render if server rendered extra nodes ([#23176](react/react#23176)) //<salazarm>//
- **[529dc3c](react/react@529dc3ce8 )**: Fix context providers in SSR when handling multiple requests ([#23171](react/react#23171)) //<Fran Dios>//
- **[505c15c](react/react@505c15c9e )**: Don't inject timeline hooks unless React supports profiling ([#23151](react/react#23151)) //<Brian Vaughn>//
- **[e12a9df](react/react@e12a9dfc9 )**: Fix production-only updateSyncExternalStore() crash when doing setState in render ([#23150](react/react#23150)) //<Douglas Armstrong>//
- **[e489402](react/react@e48940255 )**: Warn when a callback ref returns a function ([#23145](react/react#23145)) //<Dan Abramov>//
- **[d8cfeaf](react/react@d8cfeaf22 )**: Fix context propagation for offscreen/fallback trees ([#23095](react/react#23095)) //<Dan Abramov>//
- **[d504824](react/react@d50482478 )**: Enable scheduling profiler flag in react-dom/testing builds ([#23142](react/react#23142)) //<Brian Vaughn>//
- **[790b524](react/react@790b5246f )**: Fix setState ignored in Safari when iframe is added to DOM in the same commit ([#23111](react/react#23111)) //<Dan Abramov>//

Changelog:
[General][Changed] - React Native sync for revisions 51947a1...a3bde79

jest_e2e[run_all_tests]

Reviewed By: ShikaSD

Differential Revision: D34108924

fbshipit-source-id: 96acb66c1a7da79d6ef9403490cd0e29ad23d9fb
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
nevilm-lt pushed a commit to nevilm-lt/react that referenced this pull request Apr 22, 2022
@kassens
kassens deleted the revert-23236-main branch November 29, 2022 22:24
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants