Skip to content

Conversation

poteto
Copy link
Member

@poteto poteto commented Sep 26, 2024

Stack from ghstack (oldest at bottom):

Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

[ghstack-poisoned]
Copy link

vercel bot commented Sep 26, 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 Sep 27, 2024 7:16pm

Copy link
Contributor

@mofeiZ mofeiZ left a comment

Choose a reason for hiding this comment

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

Makes sense!

[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: ec2d5fb
Pull Request resolved: #31081
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: ec2d5fb
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 5cb0ccf
Pull Request resolved: #31081
@react-sizebot
Copy link

react-sizebot commented Sep 26, 2024

Comparing: 3edc000...4a0a743

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 +0.05% 1.82 kB 1.82 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 508.49 kB 508.49 kB = 90.91 kB 90.91 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 513.42 kB 513.42 kB = 91.64 kB 91.64 kB
facebook-www/ReactDOM-prod.classic.js = 603.02 kB 603.02 kB = 106.64 kB 106.64 kB
facebook-www/ReactDOM-prod.modern.js = 579.21 kB 579.21 kB = 102.72 kB 102.72 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 2583924

[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 2ae4486
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 7f4eca0
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 920f94a
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 80c9021
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 9d11a36
Pull Request resolved: #31081
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 74fe8a2
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: e206765
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: d6b0b39
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: b7aa835
Pull Request resolved: #31081
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: b7aa835
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: ab63e66
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 8157b50
Pull Request resolved: #31081
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 61a8bf9
Pull Request resolved: #31081
@poteto
Copy link
Member Author

poteto commented Sep 27, 2024

applecatrun-apple

finally

[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 56ab064
Pull Request resolved: #31081
@poteto poteto merged commit 2583924 into gh/poteto/20/base Sep 27, 2024
197 checks passed
poteto added a commit that referenced this pull request Sep 27, 2024
Currently the playground is setup as a linked workspace for the
compiler which complicates our yarn workspace setup and means that snap
can sometimes pull in a different version of react than was otherwise
specified.

There's no real reason to have these workspaces combined so let's split
them up.

ghstack-source-id: 56ab064
Pull Request resolved: #31081
@poteto poteto deleted the gh/poteto/20/head branch September 27, 2024 19:25
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.

5 participants