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

Support react@16.0.0 #31

Merged
merged 2 commits into from Nov 1, 2017

Conversation

@artkravchenko
Copy link
Contributor

artkravchenko commented Oct 15, 2017

Hello @theadam,

I have added support of usage the lib with react@16.0.0 targets. Review the changes, please.

I've also upgraded some development dependencies to run with react@16.0.0 in feature/react-16-dev. Tests are powered with updated enzyme and they pass successfully. If you'd like to upgrade them too, drop me a line - I'll update the PR.

Besides, should I target the examples to react@16.0.0 here?

Fixes #30.

@artkravchenko

This comment has been minimized.

Copy link
Contributor Author

artkravchenko commented Oct 21, 2017

Until original repository is updated to support react@16.0.0, you can use @oopscurity/react-inform npm module or fork feature/react-16 and then publish your own package.

@theadam

This comment has been minimized.

Copy link
Owner

theadam commented Nov 1, 2017

No need for the other changes, this looks great.

@theadam theadam merged commit cb1b596 into theadam:master Nov 1, 2017
@theadam theadam referenced this pull request Nov 1, 2017
@theadam

This comment has been minimized.

Copy link
Owner

theadam commented Nov 1, 2017

@Oopscurity Just released this as v0.2.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.