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

Revert "React compiler beta and reenable rule (#5898)" #6071

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Nov 1, 2024

This reverts commit 1fc684a.

Because of facebook/react#31402. While the bug itself isn't new, it manifested now because the component that triggered it was no longer bailing out. Since we don't know how many other components we have in the codebase that could trigger this bug — and how many components no longer bail out since the upgrade — the safest thing to do seems to go back to the baseline version we've been using for months. So far we haven't hit bugs with it.

Test Plan

Go to web profile > Edit > Avatar > Upload from files. It should no longer crash.

@arcalinea arcalinea temporarily deployed to revert-compiler - social-app PR #6071 November 1, 2024 21:53 — with Render Destroyed
Copy link

github-actions bot commented Nov 1, 2024

Old size New size Diff
8.14 MB 8.12 MB -21.27 KB (-0.26%)

@gaearon gaearon merged commit 7111e38 into main Nov 1, 2024
6 checks passed
@gaearon gaearon deleted the revert-compiler branch November 1, 2024 22:17
@mozzius mozzius mentioned this pull request Nov 11, 2024
gaearon added a commit that referenced this pull request Nov 17, 2024
gaearon added a commit that referenced this pull request Nov 17, 2024
* Revert "Revert "React compiler beta and reenable rule (#5898)" (#6071)"

This reverts commit 7111e38.

* Update the compiler

* Don't inline compiler runtime import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants