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

Add Redux 4 to peerDependencies #290

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Methuselah96
Copy link

No description provided.

@codecov
Copy link

codecov bot commented Jun 5, 2018

Codecov Report

Merging #290 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #290   +/-   ##
=======================================
  Coverage   98.61%   98.61%           
=======================================
  Files          28       28           
  Lines         362      362           
=======================================
  Hits          357      357           
  Misses          5        5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57a15db...bbab1a1. Read the comment docs.

@renchap
Copy link

renchap commented Sep 6, 2018

Any plans to merge this?

@renchap
Copy link

renchap commented Oct 1, 2018

@parkerziegler I saw that you released a new version, could you please merge this and release again? Redux 4.0.0 compatible with Redux 3 for user-facing API, and I have been using it with redux-little-router without issues, but this generated a warning when installing redux-little-router as the peer deps do not match.

@parkerziegler
Copy link
Contributor

Hey @renchap I can take a look at this. 15.1.2 was mostly intended as a release purely for TS defs. I'm trying to review PRs as I have time for and batch similar changes. Hopefully can get a version with this published in a few days.

@shepmaster shepmaster mentioned this pull request Dec 4, 2018
@Jameskmonger
Copy link

@parkerziegler Could this be merged and published? 😄

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.

4 participants