Skip to content

Conversation

@lunaruan
Copy link
Contributor

Modified the run_devtools_e2e_tests script so that you can pass in a React version. If you pass in a version, it will build the DevTools shell and run the e2e tests with that version.

@lunaruan lunaruan requested review from bvaughn and mondaychen May 31, 2022 16:52
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels May 31, 2022
@sizebot
Copy link

sizebot commented May 31, 2022

Comparing: f534cc6...d361359

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 = 131.54 kB 131.54 kB = 42.26 kB 42.26 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 136.80 kB 136.80 kB = 43.81 kB 43.81 kB
facebook-www/ReactDOM-prod.classic.js = 439.85 kB 439.85 kB = 80.44 kB 80.44 kB
facebook-www/ReactDOM-prod.modern.js = 425.14 kB 425.14 kB = 78.28 kB 78.28 kB
facebook-www/ReactDOMForked-prod.classic.js = 439.85 kB 439.85 kB = 80.45 kB 80.44 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against d361359

'yarn',
[
'cross-env',
'REACT_VERSION=16.8',
Copy link
Contributor

@bvaughn bvaughn May 31, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to hard-code the React version? I assume it was an accident.

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼 except for the hard-coded react version

@lunaruan lunaruan force-pushed the regression_build_script branch 2 times, most recently from a07dec6 to 88dc811 Compare May 31, 2022 17:20
@lunaruan lunaruan force-pushed the regression_build_script branch from 88dc811 to d361359 Compare May 31, 2022 18:21
@lunaruan lunaruan merged commit 0b54555 into facebook:main May 31, 2022
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