Skip to content

Fix peerDependencies#13

Merged
lttb merged 1 commit intomasterfrom
feature/fix-peer-deps
Apr 26, 2017
Merged

Fix peerDependencies#13
lttb merged 1 commit intomasterfrom
feature/fix-peer-deps

Conversation

@lttb
Copy link
Copy Markdown
Member

@lttb lttb commented Apr 26, 2017

No description provided.

@lttb lttb requested a review from kof April 26, 2017 11:05
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 90.476% when pulling f97acac on feature/fix-peer-deps into ec95181 on master.

@lttb lttb merged commit 00e06a1 into master Apr 26, 2017
@kof
Copy link
Copy Markdown
Member

kof commented Apr 26, 2017

Why older react version and why no react-dom?

@lttb
Copy link
Copy Markdown
Member Author

lttb commented Apr 26, 2017

older because people have warnings with older versions, but we require version just from '15.3.0'.

react-dom is a peer dependency of is-react-prop, but not of our package

@kof
Copy link
Copy Markdown
Member

kof commented Apr 26, 2017

right, then we might go even older

@kof
Copy link
Copy Markdown
Member

kof commented Apr 26, 2017

if there are no compatibility issues

@lttb
Copy link
Copy Markdown
Member Author

lttb commented Apr 26, 2017

not now because of PureComponent: https://github.com/facebook/react/blob/master/CHANGELOG.md#1530-july-29-2016

according to my local checks PureComponent was a little bit faster on re-render, but we need to explore this deeper of course

@lttb lttb deleted the feature/fix-peer-deps branch April 28, 2017 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants