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

[playground] Decouple playground from compiler #31081

Merged
merged 25 commits into from
Sep 27, 2024

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: ec2d5fb7721668772b76ddef36e399d09a47cf44
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: ec2d5fb7721668772b76ddef36e399d09a47cf44
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: 5cb0ccf5b463ec63c24149b18d4e23facb30b60c
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: 2ae44866104dac5f40cb19deb290280921289402
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: 7f4eca02497419be14dd2f87c92533575ddfe816
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: 920f94a07266851f85fc469acfe7837f869c1744
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: 80c90217b802c601221e2a1016c3df6ebd59db07
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: 9d11a365001134a75ed568ef7d9cb2b9d3d2528e
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: e2067656c718d8999d0cb3f817a3b4d03ef12c29
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: d6b0b3957859d9dbf9e67849af6294f0ea065abb
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: b7aa8352bee084e2ece35b45bc781518df136eb7
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: b7aa8352bee084e2ece35b45bc781518df136eb7
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: ab63e668d3e5b9466ab2e24fcfd249f44b90ff03
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: 8157b50c43d6dd314bd0647cb6f428cfc8a9346b
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: 61a8bf9e49464d140770eecf1c44201e125c3969
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: 56ab064b2fc45366f5d96d37c5d4c5dc26590234
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: 56ab064b2fc45366f5d96d37c5d4c5dc26590234
Pull Request resolved: #31081
@poteto poteto deleted the gh/poteto/20/head branch September 27, 2024 19:25
github-actions bot pushed a commit to AreaLayer/react that referenced this pull request Sep 29, 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: 56ab064b2fc45366f5d96d37c5d4c5dc26590234
Pull Request resolved: facebook#31081

DiffTrain build for [db24098](facebook@db24098)
github-actions bot pushed a commit to AreaLayer/react that referenced this pull request Sep 29, 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: 56ab064b2fc45366f5d96d37c5d4c5dc26590234
Pull Request resolved: facebook#31081

DiffTrain build for [db24098](facebook@db24098)
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