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

feat: use modern jsx transform #2619

Merged
merged 3 commits into from
Apr 16, 2024
Merged

feat: use modern jsx transform #2619

merged 3 commits into from
Apr 16, 2024

Conversation

kurtextrem
Copy link
Contributor

@kurtextrem kurtextrem commented Apr 12, 2024

This PR updates the jsx transform to the one introduced with React 17: https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html

Starting with React 19, there will be a warning when using the old one (see facebook/react#28781), plus the new transform gives a nice speed-bump with React 19: facebook/react#28781

I've also run npx react-codemod update-react-imports . to update / remove imports where not needed. I've split those into two commits so that the config changes and the autofixes can be looked at separately.

@mattgperry
Copy link
Collaborator

Great, thanks for this!

@mattgperry mattgperry added the automerge Land this PR label Apr 16, 2024
@mergetron mergetron bot merged commit 8c3a635 into main Apr 16, 2024
1 check passed
@mergetron mergetron bot deleted the feature/react-jsx branch April 16, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Land this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants