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

Deps: Upgrade react-element-to-jsx-string and react-inspector for React 18 #19104

Merged
merged 3 commits into from
Sep 5, 2022

Conversation

IanVS
Copy link
Member

@IanVS IanVS commented Sep 4, 2022

Issue: #18559 #18349

Reference: #14119

Several dependencies need to be updated to avoid peer dependency warnings for react 18.

Closes #18701
Closes #17076

What I did

I updated react-element-to-jsx-string and react-inspector to versions that support React 18. They are backwards compatible with previous versions of react as well.

How to test

CI.

@IanVS IanVS requested a review from shilman September 5, 2022 03:25
@shilman shilman changed the title Chore: Upgrade react-element-to-jsx-string and react-inspector for React 18 Deps: Upgrade react-element-to-jsx-string and react-inspector for React 18 Sep 5, 2022
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome 👏

@shilman shilman merged commit 8e00964 into next Sep 5, 2022
@shilman shilman deleted the react-18-deps branch September 5, 2022 16:21
NiGhTTraX added a commit to NiGhTTraX/ts-monorepo that referenced this pull request Nov 12, 2022
@chambo-e
Copy link

Hello 👋

It seems this PR has been included in the v7 alpha, would that be possible to cherry pick it in the 6.5 branch ?
Happy to provide a PR if so :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants