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

Upgrade devDeps to React 18 -- needs to update tests, peers, etc. #77

Closed
wants to merge 1 commit into from
Closed

Conversation

anshOFS
Copy link

@anshOFS anshOFS commented May 4, 2022

Upgrading to React 18

Upgrading to React 18
@anshOFS anshOFS closed this May 4, 2022
@anshOFS

This comment was marked as spam.

@agilgur5 agilgur5 changed the title Update package.json Upgrade devDeps to React 18 -- needs to update tests, peers, etc. May 4, 2022
@agilgur5 agilgur5 added solution: invalid scope: dependencies Pull requests that update a dependency file labels May 4, 2022
Copy link
Owner

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

Please see #76 (comment) -- there is quite a bit more that needs to change. If it were this simple, I would have upgraded it already, but unfortunately, it is not.

In particular, tests and CI will almost certainly fail on this as, per the above link, there is no Enzyme adapter for React 18. Which also means that we can't test that React 18 actually works without issues without updating the test harness.

@anshOFS anshOFS deleted the patch-1 branch May 4, 2022 16:05
@agilgur5 agilgur5 linked an issue May 4, 2022 that may be closed by this pull request
Repository owner locked as resolved and limited conversation to collaborators May 4, 2022
@agilgur5 agilgur5 added the kind: internal Changes only affect the internals and not the API or usage label Jun 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind: internal Changes only affect the internals and not the API or usage scope: dependencies Pull requests that update a dependency file solution: invalid
Projects
None yet
Development

Successfully merging this pull request may close these issues.

React 18 support in peerDeps
2 participants