Skip to content
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

Include other build types when building for CodeSandbox CI #23152

Merged
merged 1 commit into from Jan 20, 2022

Conversation

CompuIves
Copy link
Contributor

Whenever the fetching of artifacts to CircleCI times out in CodeSandbox CI, we build the packages ourselves. However, not all necessary components are in there, this adds them.

How did you test this change?

Ran yarn build-combined --type=node react/index react-dom/index react-dom/src/server react-dom/test-utils scheduler/index react/jsx-runtime react/jsx-dev-runtime to verify that all files are in there.

@sizebot
Copy link

sizebot commented Jan 20, 2022

Comparing: 05a55a4...39c9d9a

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 = 129.68 kB 129.68 kB = 41.59 kB 41.59 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 134.86 kB 134.86 kB = 43.14 kB 43.14 kB
facebook-www/ReactDOM-prod.classic.js = 428.38 kB 428.38 kB = 78.70 kB 78.70 kB
facebook-www/ReactDOM-prod.modern.js = 418.37 kB 418.37 kB = 77.27 kB 77.27 kB
facebook-www/ReactDOMForked-prod.classic.js = 428.38 kB 428.38 kB = 78.70 kB 78.70 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 39c9d9a

@gaearon
Copy link
Collaborator

gaearon commented Jan 20, 2022

ahhhhhhhhhhhhhhhhhhhhhhhhhh

@gaearon
Copy link
Collaborator

gaearon commented Jan 20, 2022

thanks ❤️

@gaearon gaearon merged commit 3fba8d6 into facebook:main Jan 20, 2022
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
nevilm-lt pushed a commit to nevilm-lt/react that referenced this pull request Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants