Skip to content
This repository has been archived by the owner on Nov 10, 2017. It is now read-only.

Add missing dependency fixes #34 #35

Merged
merged 3 commits into from
Aug 30, 2016
Merged

Add missing dependency fixes #34 #35

merged 3 commits into from
Aug 30, 2016

Conversation

thani-sh
Copy link
Contributor

Currently, react-storybook has this as a dependency.
That's why it didn't surface earlier.

@thani-sh
Copy link
Contributor Author

@arunoda shall I merge this one now? Need it for RN Storybook.

@arunoda
Copy link
Contributor

arunoda commented Aug 24, 2016

This should be a peerDependancy

@thani-sh
Copy link
Contributor Author

Yeah, just updated the PR to make it a peer dependency.

@@ -42,7 +42,8 @@
"sinon": "^1.17.3"
},
"peerDependencies": {
"react": "^0.14.7 || ^15.0.0"
"react": "^0.14.7 || ^15.0.0",
"react-dom": "^15.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add support for react-dom for 0.14.x as well.

@thani-sh
Copy link
Contributor Author

@arunoda updated the range

@arunoda
Copy link
Contributor

arunoda commented Aug 30, 2016

Go for it and do a release.

@thani-sh thani-sh merged commit 94cada8 into master Aug 30, 2016
@thani-sh thani-sh deleted the add-react-dom-dep branch August 30, 2016 10:46
@thani-sh
Copy link
Contributor Author

Released v3.3.4

ndelangen pushed a commit that referenced this pull request Apr 10, 2017
Use the branch url for notes addon
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants