petermikitsh commented 6 years ago

Summary

(If you have not already please refer to the contributing guideline as described here)

[x] Tell us about the problem your pull request is solving.

Redux gets bundled twice (wasting precious, precious bundle size bytes) if you use Redux v4 in your project, while also importing this project.

[x] Are there any open issues that are related to this?

54.

[x] Is this PR dependent on PRs in other repos?

Nope.

This is a breaking change, since it will now require users to explicitly state a version of redux to their package.json to be included. Most people using this package are probably using redux in their projects anyways... but to be safe, a major version bump is appropriate, even though none of the APIs are changing.

eddyystop commented 6 years ago

Could you bump the semver to 3.0.0 and add a "Migration to v3.0.0" section to the README? Thanks.

eddyystop commented 6 years ago

Thanks again.

Published as v3.0.0

feathersjs-ecosystem / feathers-redux

Move redux to peerDependencies (#54) #55

Summary

54.