-
Notifications
You must be signed in to change notification settings - Fork 46.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests[react-devtools]: added tests for Compiler integration #31241
tests[react-devtools]: added tests for Compiler integration #31241
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Comparing: 13411e4...16c30dc Critical size changesIncludes critical production bundles, as well as any change greater than 2%:
Significant size changesIncludes any change greater than 0.2%: (No significant changes) |
b26a2ea
to
b834e5c
Compare
b834e5c
to
bcd54a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I think given that we don't have any tests for this yet, I'm inclined to say that we ship this now and iterate. There might be a better way to handle the react-compiler-runtime package case but that doesn't need to block
) | ||
); | ||
await exec( | ||
`mv ${REGRESSION_FOLDER}/node_modules/react-compiler-runtime/dist/index.js ${buildPath}/react/compiler-runtime.js` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems confusing and maybe unnecessary to overrwrite react/compiler-runtime.js, but I see why it's useful to
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to overrwrite react/compiler-runtime.js
react/compiler-runtime.js
is only shipped in React 19, and this check is gated for React [18.2, 19), so we don't override anything, just copying a script to replicate same react
package structure
Adds tests for Compiler integration.
This includes:
For tests against React 18.2, I had to download
react-compiler-runtime
from npm and put it toreact/compiler-runtime.js
.