Skip to content

Conversation

kassens
Copy link
Member

@kassens kassens commented Oct 4, 2022

Stack from ghstack (oldest at bottom):

This upgrade made more expressions invalidate refinements. In some
places this lead to a large number of suppressions that I automatically
suppressed and should be followed up on when the code is touched.
I think most of them might require either manual annotations or moving
a value into a const to allow refinement.

This upgrade made more expressions invalidate refinements. In some
places this lead to a large number of suppressions that I automatically
suppressed and should be followed up on when the code is touched.
I think most of them might require either manual annotations or moving
a value into a const to allow refinement.

[ghstack-poisoned]
This was referenced Oct 4, 2022
kassens added a commit that referenced this pull request Oct 4, 2022
This upgrade made more expressions invalidate refinements. In some
places this lead to a large number of suppressions that I automatically
suppressed and should be followed up on when the code is touched.
I think most of them might require either manual annotations or moving
a value into a const to allow refinement.

ghstack-source-id: a45b40a
Pull Request resolved: #25410
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Oct 4, 2022
@sizebot
Copy link

sizebot commented Oct 4, 2022

Comparing: 0a30722...2bec0fd

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.min.js = 135.46 kB 135.46 kB = 43.41 kB 43.41 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 147.72 kB 147.72 kB = 47.17 kB 47.17 kB
facebook-www/ReactDOM-prod.classic.js = 491.70 kB 491.70 kB = 87.49 kB 87.49 kB
facebook-www/ReactDOM-prod.modern.js = 477.00 kB 477.00 kB = 85.24 kB 85.24 kB
facebook-www/ReactDOMForked-prod.classic.js = 491.70 kB 491.70 kB = 87.49 kB 87.49 kB
facebook-www/SchedulerMock-dev.classic.js +4.04% 23.29 kB 24.23 kB +1.40% 5.56 kB 5.64 kB
facebook-www/SchedulerMock-dev.modern.js +4.04% 23.29 kB 24.23 kB +1.40% 5.56 kB 5.64 kB
oss-experimental/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-stable-semver/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-stable/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-experimental/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
oss-stable-semver/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
oss-stable/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
facebook-react-native/scheduler/cjs/SchedulerMock-dev.js +3.42% 18.06 kB 18.68 kB +1.44% 4.37 kB 4.43 kB
oss-experimental/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
oss-stable-semver/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
oss-stable/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
facebook-www/Scheduler-dev.classic.js +3.22% 25.18 kB 25.99 kB +1.14% 6.76 kB 6.84 kB
facebook-www/Scheduler-dev.modern.js +3.22% 25.18 kB 25.99 kB +1.14% 6.76 kB 6.84 kB
oss-experimental/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
oss-stable-semver/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
oss-stable/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
facebook-react-native/scheduler/cjs/Scheduler-dev.js +2.72% 17.93 kB 18.42 kB +0.91% 5.07 kB 5.11 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-www/SchedulerMock-dev.classic.js +4.04% 23.29 kB 24.23 kB +1.40% 5.56 kB 5.64 kB
facebook-www/SchedulerMock-dev.modern.js +4.04% 23.29 kB 24.23 kB +1.40% 5.56 kB 5.64 kB
oss-experimental/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-stable-semver/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-stable/react-fs/cjs/react-fs.node.development.server.js +3.58% 10.05 kB 10.41 kB +1.86% 2.47 kB 2.52 kB
oss-experimental/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
oss-stable-semver/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
oss-stable/scheduler/cjs/scheduler-unstable_mock.development.js +3.43% 18.01 kB 18.62 kB +1.42% 4.37 kB 4.43 kB
facebook-react-native/scheduler/cjs/SchedulerMock-dev.js +3.42% 18.06 kB 18.68 kB +1.44% 4.37 kB 4.43 kB
oss-experimental/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
oss-stable-semver/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
oss-stable/scheduler/umd/scheduler-unstable_mock.development.js +3.25% 19.35 kB 19.98 kB +1.27% 4.48 kB 4.54 kB
facebook-www/Scheduler-dev.classic.js +3.22% 25.18 kB 25.99 kB +1.14% 6.76 kB 6.84 kB
facebook-www/Scheduler-dev.modern.js +3.22% 25.18 kB 25.99 kB +1.14% 6.76 kB 6.84 kB
oss-experimental/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
oss-stable-semver/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
oss-stable/scheduler/cjs/scheduler.development.js +2.73% 17.90 kB 18.39 kB +0.97% 5.06 kB 5.10 kB
facebook-react-native/scheduler/cjs/Scheduler-dev.js +2.72% 17.93 kB 18.42 kB +0.91% 5.07 kB 5.11 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-plugin.js +1.52% 9.37 kB 9.51 kB +1.42% 2.96 kB 3.00 kB
oss-stable-semver/react-server-dom-webpack/cjs/react-server-dom-webpack-plugin.js +1.52% 9.37 kB 9.51 kB +1.42% 2.96 kB 3.00 kB
oss-stable/react-server-dom-webpack/cjs/react-server-dom-webpack-plugin.js +1.52% 9.37 kB 9.51 kB +1.42% 2.96 kB 3.00 kB
oss-experimental/react-dom/cjs/react-dom-static.node.development.js +0.55% 300.30 kB 301.95 kB +0.29% 71.45 kB 71.66 kB
oss-experimental/react-dom/cjs/react-dom-server.node.development.js +0.55% 300.34 kB 301.99 kB +0.29% 71.36 kB 71.57 kB
oss-experimental/react-dom/cjs/react-dom-static.browser.development.js +0.54% 298.40 kB 300.00 kB +0.25% 71.25 kB 71.42 kB
oss-experimental/react-dom/cjs/react-dom-server.browser.development.js +0.53% 299.10 kB 300.70 kB +0.24% 71.44 kB 71.62 kB
oss-experimental/react-dom/cjs/react-dom-server-legacy.browser.development.js +0.53% 300.16 kB 301.76 kB +0.25% 71.37 kB 71.55 kB
oss-experimental/react-dom/cjs/react-dom-server-legacy.node.development.js +0.53% 301.87 kB 303.46 kB +0.25% 71.84 kB 72.02 kB
oss-experimental/react-dom/umd/react-dom-server.browser.development.js +0.52% 313.69 kB 315.31 kB +0.27% 72.17 kB 72.36 kB
oss-experimental/react-dom/umd/react-dom-server-legacy.browser.development.js +0.51% 314.77 kB 316.39 kB +0.26% 72.13 kB 72.31 kB
oss-stable-semver/react/umd/react.development.js +0.50% 110.81 kB 111.37 kB +0.21% 28.52 kB 28.58 kB
oss-stable/react/umd/react.development.js +0.50% 110.84 kB 111.40 kB +0.20% 28.54 kB 28.60 kB
oss-experimental/react/umd/react.development.js +0.48% 115.25 kB 115.81 kB +0.18% 29.58 kB 29.63 kB
oss-stable-semver/react-server/cjs/react-server.development.js +0.44% 137.10 kB 137.71 kB +0.20% 34.40 kB 34.47 kB
oss-stable/react-server/cjs/react-server.development.js +0.44% 137.10 kB 137.71 kB +0.20% 34.40 kB 34.47 kB
oss-experimental/react-server/cjs/react-server.development.js +0.42% 143.22 kB 143.82 kB +0.22% 35.77 kB 35.85 kB
oss-stable-semver/react-dom/cjs/react-dom-server.node.development.js +0.38% 279.16 kB 280.21 kB +0.26% 66.93 kB 67.11 kB
oss-stable/react-dom/cjs/react-dom-server.node.development.js +0.38% 279.18 kB 280.23 kB +0.26% 66.96 kB 67.13 kB
oss-stable-semver/react-dom/cjs/react-dom-server-legacy.browser.development.js +0.36% 277.60 kB 278.59 kB +0.22% 66.46 kB 66.61 kB
oss-stable/react-dom/cjs/react-dom-server-legacy.browser.development.js +0.36% 277.62 kB 278.62 kB +0.22% 66.49 kB 66.63 kB
oss-stable-semver/react-dom/cjs/react-dom-server.browser.development.js +0.36% 277.92 kB 278.91 kB +0.22% 67.01 kB 67.15 kB
oss-stable/react-dom/cjs/react-dom-server.browser.development.js +0.36% 277.94 kB 278.94 kB +0.21% 67.03 kB 67.17 kB
oss-stable-semver/react-dom/cjs/react-dom-server-legacy.node.development.js +0.35% 279.31 kB 280.30 kB +0.22% 66.94 kB 67.08 kB
oss-stable/react-dom/cjs/react-dom-server-legacy.node.development.js +0.35% 279.33 kB 280.32 kB +0.21% 66.96 kB 67.10 kB
facebook-www/ReactDOMServerStreaming-dev.modern.js +0.35% 284.67 kB 285.66 kB +0.19% 67.21 kB 67.34 kB
oss-stable-semver/react-dom/umd/react-dom-server-legacy.browser.development.js +0.34% 291.06 kB 292.07 kB +0.22% 67.15 kB 67.30 kB
oss-stable/react-dom/umd/react-dom-server-legacy.browser.development.js +0.34% 291.09 kB 292.09 kB +0.22% 67.18 kB 67.32 kB
oss-stable-semver/react-dom/umd/react-dom-server.browser.development.js +0.34% 291.38 kB 292.38 kB +0.21% 67.69 kB 67.83 kB
oss-stable/react-dom/umd/react-dom-server.browser.development.js +0.34% 291.40 kB 292.41 kB +0.21% 67.71 kB 67.85 kB
facebook-www/ReactDOMServer-dev.modern.js +0.34% 289.17 kB 290.15 kB +0.19% 68.21 kB 68.34 kB
facebook-www/ReactDOMServer-dev.classic.js +0.33% 295.87 kB 296.85 kB +0.19% 69.68 kB 69.81 kB

Generated by 🚫 dangerJS against 2bec0fd

@kassens kassens merged commit 2bec0fd into gh/kassens/3/base Oct 4, 2022
kassens added a commit that referenced this pull request Oct 4, 2022
This upgrade made more expressions invalidate refinements. In some
places this lead to a large number of suppressions that I automatically
suppressed and should be followed up on when the code is touched.
I think most of them might require either manual annotations or moving
a value into a const to allow refinement.

ghstack-source-id: a45b40a
Pull Request resolved: #25410
@kassens kassens deleted the gh/kassens/3/head branch October 4, 2022 15:02
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