-
Notifications
You must be signed in to change notification settings - Fork 13
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
React 18 support #109
Comments
Sure! Would you like the package version to be bumped to 18.0.0 as well? |
I don't work on react anymore and can't speak for the team, but perhaps @gaearon has an opinion? |
We didn’t bump it to 17 so maybe let’s not bump to 18 either? So that it doesn’t seem like it’s part of the release or encouraged for use. We should probably shim the new Hooks though before the bump (to 16.15 or something). There aren’t that many and implementations should be simple. |
Could you elaborate on what's required here? But I think you should still have npm and GitHub access so feel free to make the necessary changes yourself |
I mean providing some shims for useId etc. |
Fixed in [email protected]. |
In my very very limited use, it doesn't appear to be a problem to just be using React 18 so far, but it would be good to explicitly add 18 to peerDeps if compatible. This would just silence the install time warning.
The text was updated successfully, but these errors were encountered: