-
Notifications
You must be signed in to change notification settings - Fork 115
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
Add children to the props for @types/react@18 #192
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, low risk
Any update here? No way to switch to React 18 without this fix |
Can we get this merged? |
I am unfortunately not an admin on this project anymore, so I am unable to bypass a broken CI check |
Any update on this ? it would be nice if could merge this PR. |
Can this get merged and released? Waiting on this fix. |
Any updates on when this will be merged? |
In relation to React props should not do anything special with
children
and DefinitelyTyped have updated the react types to v18DefinitelyTyped/DefinitelyTyped#59802