Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Removed redux from peer dependencies for v2 #1224

Conversation

jaydenseric
Copy link
Contributor

Fixes #1223.

Checklist:

  • If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
  • Make sure all of the significant new logic is covered by tests
  • If this was a change that affects the external API used in GitHunt-React, update GitHunt-React and post a link to the PR in the discussion.

@jaydenseric jaydenseric changed the title Removed redux from peer dependencies. Removed redux from peer dependencies for v2 Oct 11, 2017
@jbaxleyiii jbaxleyiii merged commit 37d432c into apollographql:apollo-client-2.0 Oct 15, 2017
@@ -83,8 +83,7 @@
"license": "MIT",
"peerDependencies": {
"apollo-client": "^2.0.0-beta.3",
"react": "0.14.x || 15.* || ^15.0.0 || ^16.0.0",
"redux": "^2.0.0 || ^3.0.0"
"react": "0.14.x || 15.* || ^15.0.0 || ^16.0.0"
Copy link

Choose a reason for hiding this comment

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

Is there any difference between 15.* and ^15.0.0?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No difference, it looks redundant.

I think:

"react": "0.14.x || 15.* || ^15.0.0 || ^16.0.0"

Can be simplified to:

"react": "0.14 - 16"

@jaydenseric jaydenseric deleted the remove-redux-peer-dependency branch January 29, 2018 12:00
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