Skip to content

[ci] Parallelize yarn build #30071

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

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: 636f2bb
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: 4cf4021
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: d14f9ea
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 3cb64db
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 33b498f
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: 0767a1f
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: 2beffba
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jun 24, 2024
ghstack-source-id: fc61ba2
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added 3 commits June 24, 2024 17:23
[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 that referenced this pull request Jul 11, 2024
ghstack-source-id: 78cf030
Pull Request resolved: #30071
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 11, 2024
ghstack-source-id: c93d81e
Pull Request resolved: #30071
poteto added a commit that referenced this pull request Jul 12, 2024
ghstack-source-id: c93d81e
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
Member

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
Member

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: 8a13b45
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