Skip to content

Conversation

@poteto
Copy link
Member

@poteto poteto commented Mar 19, 2025

@react-sizebot
Copy link

react-sizebot commented Mar 19, 2025

Comparing: 9954104...cb656cd

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 = 517.29 kB 517.29 kB = 92.26 kB 92.26 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 = 608.06 kB 608.06 kB = 107.88 kB 107.88 kB
facebook-www/ReactDOM-prod.classic.js = 653.67 kB 653.67 kB = 115.18 kB 115.18 kB
facebook-www/ReactDOM-prod.modern.js = 643.95 kB 643.95 kB = 113.59 kB 113.59 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against cb656cd

No reason to download them from scratch every time.
@poteto poteto merged commit a8c155c into main Mar 19, 2025
206 checks passed
@poteto poteto deleted the pr32678 branch March 19, 2025 21:22
poteto added a commit that referenced this pull request Mar 19, 2025
Defaults to warn, but since some steps require these artifacts to be
uploaded we specify an error if its not found. Some other steps like
playwright test-results are only uploaded on failure so it's okay to
ignore.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/32679).
* #32680
* __->__ #32679
* #32678
poteto added a commit that referenced this pull request Mar 19, 2025
There was a bug previously in our commit artifacts step where the
emitted REVISION hash would reference the commit on the builds branch
rather than from `main`.

Given that our internal manual sync script also does this, let's align
them both to always reference the commit from `main` instead.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/32680).
* __->__ #32680
* #32679
* #32678
poteto added a commit that referenced this pull request Mar 20, 2025
Following #32678, do the same for the playground e2e test since this step can sometimes take many minutes to complete.
poteto added a commit that referenced this pull request Mar 20, 2025
Following #32678, do the same for the playground e2e test since this step can sometimes take many minutes to complete.
poteto added a commit that referenced this pull request Mar 20, 2025
Following #32678, do the same for the playground e2e test since this step can sometimes take many minutes to complete.
poteto added a commit that referenced this pull request Mar 20, 2025
Following #32678, do the same for the playground e2e test since this step can sometimes take many minutes to complete.
poteto added a commit that referenced this pull request Mar 20, 2025
Following #32678, do the same for the playground e2e test since this
step can sometimes take many minutes to complete.
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