Skip to content

Conversation

rickhanlonii
Copy link
Member

Will run a quick experiment for this.

@rickhanlonii rickhanlonii marked this pull request as ready for review December 16, 2024 15:29
Copy link

vercel bot commented Dec 16, 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 16, 2024 3:40pm

@facebook-github-bot facebook-github-bot added the React Core Team Opened by a member of the React Core Team label Dec 16, 2024
@react-sizebot
Copy link

react-sizebot commented Dec 16, 2024

Comparing: e06c72f...e9d7187

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 +0.05% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 510.76 kB 510.76 kB = 91.36 kB 91.36 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 515.55 kB 515.55 kB = 92.17 kB 92.17 kB
facebook-www/ReactDOM-prod.classic.js = 594.69 kB 594.69 kB = 104.93 kB 104.93 kB
facebook-www/ReactDOM-prod.modern.js = 584.96 kB 584.96 kB = 103.37 kB 103.37 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-www/Scheduler-prod.classic.js +1.22% 10.94 kB 11.07 kB +0.93% 2.69 kB 2.71 kB
facebook-www/Scheduler-prod.modern.js +1.22% 10.94 kB 11.07 kB +0.93% 2.69 kB 2.71 kB
facebook-www/Scheduler-profiling.classic.js +1.17% 11.37 kB 11.51 kB +0.93% 2.79 kB 2.81 kB
facebook-www/Scheduler-profiling.modern.js +1.17% 11.37 kB 11.51 kB +0.93% 2.79 kB 2.81 kB
facebook-www/Scheduler-dev.classic.js +0.83% 16.52 kB 16.66 kB +0.60% 3.51 kB 3.53 kB
facebook-www/Scheduler-dev.modern.js +0.83% 16.52 kB 16.66 kB +0.60% 3.51 kB 3.53 kB

Generated by 🚫 dangerJS against 8c7345f

@rickhanlonii rickhanlonii merged commit f7b1273 into facebook:main Dec 16, 2024
187 checks passed
@rickhanlonii rickhanlonii deleted the rh/ff-request-paint branch December 16, 2024 16:18
github-actions bot pushed a commit that referenced this pull request Dec 16, 2024
Will run a quick experiment for this.

DiffTrain build for [f7b1273](f7b1273)
davesnx added a commit to davesnx/react that referenced this pull request Dec 18, 2024
…act into issue-31578-setter-arity-gcc

* 'issue-31578-setter-arity-gcc' of github.com:/davesnx/react:
  [flags] Clean up scheduler flags (facebook#31814)
  Enable debugRenderPhaseSideEffectsForStrictMode in test renderers (facebook#31761)
  Enable disableDefaultPropsExceptForClasses (facebook#31804)
  Turn on useModernStrictMode in test renderers (facebook#31769)
  [compiler][ez] Add shape for global Object.keys (facebook#31583)
  [compiler] Context variables as dependencies (facebook#31582)
  [compiler] Add fire to known React APIs (facebook#31795)
  [compiler] Add option for firing effect functions (facebook#31794)
  [compiler][be] Logger based debug printing in test runner (facebook#31809)
  [compiler][ez] Clean up duplicate code in propagateScopeDeps (facebook#31581)
  [compiler] Repro for aliased captures within inner function expressions (facebook#31770)
  [compiler][be] Playground now compiles entire program (facebook#31774)
  [Flight] Color and badge non-primary environments (facebook#31738)
  [Flight] Emit Deduped Server Components Marker (facebook#31737)
  [Flight] Sort Server Components Track Group ahead of Client Scheduler/Components Tracks (facebook#31736)
  Clean up context access profiling experiment (facebook#31806)
  [Flight] Stack Parallel Components in Separate Tracks (facebook#31735)
  Flag for requestPaint (facebook#31805)
jackpope added 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.
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