-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 16 #480
React 16 #480
Conversation
} | ||
}, | ||
"setupFiles": [ | ||
"raf/polyfill", |
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.
Should be able to remove this by updating to the latest version of jest. See jestjs/jest#4568
I am going to attempt it and if it is smooth I will remove this.
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.
This hasn't released yet. 😅 The upgrade to jest@21
was painless though.
Looks great! 👍 Thanks Kyle! |
@coryhouse @kwelch Was about to ask if slingshot can be updated to react 16 and saw this 😄 |
Yep, 16 is in. :) |
Hi. I've started off the project with this template a year ago and it's very outdated. We plan to upgrade but not sure where to start. Could you let me know how to upgrade to the latest? Here is my current package.json.
Do I just need to update the packages.json with your new packages.json? |
The easiest way to assure you get all the updates is to paste your app code into a new copy of react-slingshot. |
This was the minimum I could do to update to React 16.
The other PR included all packages updated as well. Once this is merged I will proceed with a package update across the board.