Skip to content

Conversation

@poteto
Copy link
Member

@poteto poteto commented Jul 19, 2024

Stack from ghstack (oldest at bottom):

More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

[ghstack-poisoned]
@vercel
Copy link

vercel bot commented Jul 19, 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 25, 2024 9:17pm

poteto added a commit that referenced this pull request Jul 19, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: c60f15f
Pull Request resolved: #30406
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jul 19, 2024
@react-sizebot
Copy link

react-sizebot commented Jul 19, 2024

Comparing: 0a76aec...9a7d2f4

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.82 kB 1.82 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 501.39 kB 501.39 kB = 89.96 kB 89.96 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 = 506.21 kB 506.21 kB = 90.66 kB 90.66 kB
facebook-www/ReactDOM-prod.classic.js = 598.83 kB 598.83 kB = 105.76 kB 105.76 kB
facebook-www/ReactDOM-prod.modern.js = 574.94 kB 574.94 kB = 101.99 kB 101.99 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 97bc220

[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 19, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: e76d824
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 19, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: d33946f
Pull Request resolved: #30406
@hoxyq
Copy link
Contributor

hoxyq commented Jul 19, 2024

I have no idea why e2e tests are failing for React 16, because they were passing on main yesterday and there were no changes to React / React DevTools since then. Maybe there is something related to the environment, is the version of Node the same?

[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 19, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 90332d0
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 22, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 811e30d
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 22, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 2e9b440
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 22, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 2bf2e60
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 22, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 811e30d
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 25, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 5e3c190
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 25, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: 32fcfdc
Pull Request resolved: #30406
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 25, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: cb3456c
Pull Request resolved: #30406
@poteto
Copy link
Member Author

poteto commented Jul 25, 2024

@poteto poteto requested a review from hoxyq July 25, 2024 21:25
@poteto poteto merged commit 97bc220 into gh/poteto/34/base Jul 25, 2024
poteto added a commit that referenced this pull request Jul 25, 2024
More or less a straight copy from the circleci config. I spotted some
inefficiencies but will fix those later to make reviewing this easier.

ghstack-source-id: cb3456c
Pull Request resolved: #30406
@poteto poteto deleted the gh/poteto/34/head branch July 25, 2024 21:27
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