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

[rcr] Remove runtimeModule compiler option #31145

Merged
merged 23 commits into from
Oct 7, 2024
Merged

[rcr] Remove runtimeModule compiler option #31145

merged 23 commits into from
Oct 7, 2024

Conversation

poteto
Copy link
Member

@poteto poteto commented Oct 7, 2024

Now that the compiler always injects react-compiler-runtime, this
option is unnecessary.

poteto added 7 commits October 7, 2024 15:18
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@poteto
Copy link
Member Author

poteto commented Oct 7, 2024

Stack from ghstack (oldest at bottom):

Copy link

vercel bot commented Oct 7, 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 Oct 7, 2024 10:01pm

poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 8141f6dfef33cd85b6bee93926fde53f8c9f2d0f
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Oct 7, 2024
poteto added 4 commits October 7, 2024 15:20
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 8141f6dfef33cd85b6bee93926fde53f8c9f2d0f
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
poteto added 3 commits October 7, 2024 15:22
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: ec5ff3eb0bfd74ff9b96214df4601d417aae71c4
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
poteto added 2 commits October 7, 2024 15:30
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 6ad6822793525e72635de3aec5caeb232d308ff3
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
poteto added 6 commits October 7, 2024 16:25
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 6ad6822793525e72635de3aec5caeb232d308ff3
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
@poteto poteto force-pushed the gh/poteto/30/head branch from 0d8cf4b to c2c5431 Compare October 7, 2024 21:51
Base automatically changed from gh/poteto/30/head to main October 7, 2024 21:59
[ghstack-poisoned]
poteto added a commit that referenced this pull request Oct 7, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 6ad6822793525e72635de3aec5caeb232d308ff3
ghstack-comment-id: 2397700297
Pull Request resolved: #31145
@poteto poteto merged commit 23cd3ac into main Oct 7, 2024
19 checks passed
@poteto poteto deleted the gh/poteto/31/head branch October 7, 2024 22:07
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.

2 participants