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

using recommended packages to avoid deprecation warnings #48

Merged
merged 1 commit into from
Aug 20, 2017

Conversation

alexreardon
Copy link
Collaborator

@alexreardon alexreardon commented Aug 20, 2017

This is in anticipation of #8
This removes all the yuck deprecation warnings in the tests and development environments.

It also opens the way to React 16! I bumped to React 16 and everything looks good! I will not to React 16 until it is stable - but it good to know we are ready to go

(how I checked react 16: facebook/react#10294)

It was not a super in depth analysis - but it looks good on initial pass.

Some of the tests break moving to React 16 - but it looks like jest was throwing and error because the enzyme React dependency was not met correct - which makes sense.

@alexreardon alexreardon merged commit 0f557e0 into master Aug 20, 2017
@alexreardon alexreardon deleted the using-correct-imports branch August 20, 2017 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant