-
Notifications
You must be signed in to change notification settings - Fork 26.7k
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
update devdDependencies @types/react and @types/react-dom #37423
update devdDependencies @types/react and @types/react-dom #37423
Conversation
You can't install React 18 defs if the project is using React 17. |
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.
Thanks, could we upgrade all other dependencies as well? 👍
@saurabhmehta1601 please address my comment here, so we can merge this PR: #37423 (review) |
Thanks @balazsorban44 for the review . I am new to contributing to large codebases . Sorry if I have unknowingly caused any inconvenience in the process . |
No problem @saurabhmehta1601! I'm happy to help your first PR merged to the Next.js repo! What I meant is this: next.js/examples/with-redux/package.json Lines 13 to 14 in 756dc8f
At the least, these should be upgraded as well so the |
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.
Thank you!
fixes #37421