Skip to content
This repository was archived by the owner on Dec 15, 2018. It is now read-only.

Upgrades the peer dependency for react-redux to 5.0 (optionally 4) #134

Merged
merged 1 commit into from
Jan 17, 2017

Conversation

adregan
Copy link
Contributor

@adregan adregan commented Jan 17, 2017

I have been seeing a peer dependency warning re. react-redux, and this PR bumps the version but maintains the older version (as established with the other peer dependencies). After a period of time—once it's fairly certain that 5.0 is in actuality a non-breaking release—the 4.0 option could be removed.

From the commit:

Though react-redux's [5.0.0 release is backwards compatible with 4.0.0])(https://github.com/reactjs/react-redux/releases/tag/v5.0.0), in the spirit of supporting multiple versions as with the react and react-dom dependency, this change allows either version to be used.

This will silence the peer dependency warning.

Though react-redux's [5.0.0 release is backwards compatible with 4.0.0])(https://github.com/reactjs/react-redux/releases/tag/v5.0.0), in the spirit of supporting multiple versions as with the react and react-dom dependency, this change allows either version to be used. 

This will silence the peer dependency warning.
@codecov-io
Copy link

Current coverage is 98.26% (diff: 100%)

Merging #134 into master will not change coverage

@@             master       #134   diff @@
==========================================
  Files            19         19          
  Lines           231        231          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            227        227          
  Misses            4          4          
  Partials          0          0          

Powered by Codecov. Last update 74fcd4c...de2eb3d

Copy link
Contributor

@tptee tptee left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for doing this!

@tptee tptee merged commit 05ec969 into FormidableLabs:master Jan 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants