Skip to content

[rcr] Remove runtimeModule compiler option #31055

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

Closed
wants to merge 16 commits into from

Conversation

[ghstack-poisoned]
Copy link

vercel bot commented Sep 24, 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 2, 2024 6:59pm

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

ghstack-source-id: f0c73da
Pull Request resolved: #31055
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Sep 24, 2024
poteto added a commit that referenced this pull request Sep 24, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: f0c73da
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 24, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: b6990b2
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 0c65d96
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 159d6c4
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 85e32a6
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 26, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: c12286c
Pull Request resolved: #31055
[ghstack-poisoned]
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: b8d31c1
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: d0c9cc9
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 788fd4f
Pull Request resolved: #31055
[ghstack-poisoned]
poteto added a commit that referenced this pull request Sep 27, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 93379c6
Pull Request resolved: #31055
@josephsavona
Copy link
Member

Hmmm feels like we might want to keep this option for use internally where we're always on latest?

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

ghstack-source-id: d8f7403
Pull Request resolved: #31055
@poteto
Copy link
Member Author

poteto commented Oct 1, 2024

@josephsavona hmm the new react-compiler-runtime already defers to the latest impl, and with this stack it's always added as a dep in codegen so this option isn't really necessary anymore

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

ghstack-source-id: 7ce2b97
Pull Request resolved: #31055
poteto added a commit that referenced this pull request Oct 1, 2024
Now that the compiler always injects `react-compiler-runtime`, this
option is unnecessary.

ghstack-source-id: 7ce2b97
Pull Request resolved: #31055
[ghstack-poisoned]
@poteto poteto marked this pull request as ready for review October 1, 2024 18:27
[ghstack-poisoned]
[ghstack-poisoned]
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.

3 participants