Skip to content

Conversation

jackpope
Copy link
Contributor

#31787 introduces an experimental scheduler flag: enableAlwaysYieldScheduler, which is turned off for www. There wasn't a SchedulerFeatureFlags fork for native-fb, so the experimental change was enabled in the Scheduler-dev build there which causes test failures and is blocking the sync.

#31805 introduces another scheduler flag enableRequestPaint, which is set as a __VARIANT__ on www. I've set this to true here to preserve the existing behavior. We can follow up with dynamic flags for native-fb after unblocking the sync.

Copy link

vercel bot commented Dec 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2024 4:14pm

@react-sizebot
Copy link

Comparing: 9463d51...b90a0f3

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 511.38 kB 511.38 kB = 91.38 kB 91.38 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 517.53 kB 517.53 kB = 92.34 kB 92.34 kB
facebook-www/ReactDOM-prod.classic.js = 593.09 kB 593.09 kB = 104.45 kB 104.45 kB
facebook-www/ReactDOM-prod.modern.js = 583.35 kB 583.35 kB = 102.90 kB 102.90 kB
facebook-react-native/scheduler/cjs/Scheduler-dev.js +31.34% 11.91 kB 15.65 kB +27.43% 2.61 kB 3.33 kB
facebook-react-native/scheduler/cjs/SchedulerMock-dev.js +24.67% 13.64 kB 17.00 kB +23.48% 2.89 kB 3.57 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-react-native/scheduler/cjs/Scheduler-dev.js +31.34% 11.91 kB 15.65 kB +27.43% 2.61 kB 3.33 kB
facebook-react-native/scheduler/cjs/SchedulerMock-dev.js +24.67% 13.64 kB 17.00 kB +23.48% 2.89 kB 3.57 kB
facebook-react-native/scheduler/cjs/Scheduler-prod.js +1.33% 10.04 kB 10.18 kB +1.95% 2.46 kB 2.51 kB

Generated by 🚫 dangerJS against 64facf9

@jackpope jackpope merged commit bd76ce5 into facebook:main Dec 19, 2024
187 checks passed
@jackpope jackpope deleted the fork-scheduler-native-fb branch December 19, 2024 16:49
github-actions bot pushed a commit that referenced this pull request Dec 19, 2024
#31787 introduces an experimental scheduler flag:
`enableAlwaysYieldScheduler`, which is turned off for www. There wasn't
a SchedulerFeatureFlags fork for native-fb, so the experimental change
was enabled in the Scheduler-dev build there which causes test failures
and is blocking the sync.

#31805 introduces another scheduler flag `enableRequestPaint`, which is
set as a `__VARIANT__` on www. I've set this to `true` here to preserve
the existing behavior. We can follow up with dynamic flags for native-fb
after unblocking the sync.

DiffTrain build for [bd76ce5](bd76ce5)
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.

4 participants