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

[ci] Parallelize yarn build #30071

Merged
merged 31 commits into from
Jul 12, 2024
Merged

[ci] Parallelize yarn build #30071

merged 31 commits into from
Jul 12, 2024

Conversation

[ghstack-poisoned]
Copy link

vercel bot commented Jun 24, 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 Jul 12, 2024 3:11pm

poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 636f2bb2ade35cf648c422de29a04fee4697db26
Pull Request resolved: #30071
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jun 24, 2024
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 4cf40214f8ec1e6dcd4b1941bc2fb5b59bcc1004
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: d14f9ea5349ce08203a6b968f27736d8983d9f5f
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 3cb64db93d641d7f4a07b6321ecdb3b83b1a68ce
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 33b498fd49f158d6947b4e6b38a99452328e59ef
Pull Request resolved: #30071
@react-sizebot
Copy link

react-sizebot commented Jun 24, 2024

Comparing: e6783e7...72a15f5

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 = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 497.99 kB 497.99 kB = 89.27 kB 89.27 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 502.81 kB 502.81 kB = 89.97 kB 89.97 kB
facebook-www/ReactDOM-prod.classic.js = 597.08 kB 597.08 kB = 105.33 kB 105.33 kB
facebook-www/ReactDOM-prod.modern.js = 571.42 kB 571.42 kB = 101.27 kB 101.27 kB
test_utils/ReactAllWarnings.js Deleted 62.88 kB 0.00 kB Deleted 15.69 kB 0.00 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
test_utils/ReactAllWarnings.js Deleted 62.88 kB 0.00 kB Deleted 15.69 kB 0.00 kB

Generated by 🚫 dangerJS against 72a15f5

[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 0767a1f89b5f6bac26a699dd5591af7e0c93ed3a
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 2beffbad66dd845d58b605419571c2faa7ee0cac
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: fc61ba266b164d9bf3831d4d6e37da261481a916
Pull Request resolved: #30071
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@poteto poteto changed the title [ci] Parallelize yarn build and yarn lint-build [ci] Parallelize yarn build Jul 10, 2024
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit to poteto/react that referenced this pull request Jul 10, 2024
ghstack-source-id: 78cf0304633fc31f2372c683258b543ddd20a7af
Pull Request resolved: facebook#30071
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: 78cf0304633fc31f2372c683258b543ddd20a7af
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e370edf1ec8f8e54470c01e63f8280e542
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e370edf1ec8f8e54470c01e63f8280e542
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e370edf1ec8f8e54470c01e63f8280e542
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 12, 2024
ghstack-source-id: c93d81e370edf1ec8f8e54470c01e63f8280e542
Pull Request resolved: #30071
[ghstack-poisoned]
Copy link
Member Author

Choose a reason for hiding this comment

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

I will unfork this later

@@ -0,0 +1,869 @@
'use strict';
Copy link
Contributor

Choose a reason for hiding this comment

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

Review note: diffed this against rollup/build.js locally. Its mostly a copy with some changes to how we get node count from the runner. LGTM, assume we'll remove rollup/build.js once this is stable.

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah I'm gonna remove the circleci hardcoding later in the stack and unfork these files

strategy:
matrix:
# yml is dumb. update the --total arg to yarn build if you change the number of workers
worker_id: [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19]
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious why 20 workers. Does this match the number of bundles?

Copy link
Member Author

Choose a reason for hiding this comment

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

I preserved our current circleci setup, where we have 20 workers for stable and 20 workers for experimental. That's the same as what this config in GH expands to

@poteto poteto merged commit a52cbbc into gh/poteto/25/base Jul 12, 2024
181 checks passed
poteto added a commit that referenced this pull request Jul 12, 2024
ghstack-source-id: 8a13b456f1638a44c6f960c44f5752e8e4d32507
Pull Request resolved: #30071
@poteto poteto deleted the gh/poteto/25/head branch July 12, 2024 16:09
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