Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Framework: Update to redux 4.0.0 #25038

Merged
merged 1 commit into from
May 28, 2018
Merged

Framework: Update to redux 4.0.0 #25038

merged 1 commit into from
May 28, 2018

Conversation

blowery
Copy link
Contributor

@blowery blowery commented May 24, 2018

Moves us up to the current redux.

Breaking Changes: reduxjs/redux#1342
Release Notes: https://github.com/reduxjs/redux/releases/tag/v4.0.0

@blowery blowery requested review from ockham, sirreal, gwwar and a team May 24, 2018 14:02
@blowery blowery self-assigned this May 24, 2018
@blowery blowery added Framework [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] Needs e2e Testing labels May 24, 2018
Copy link
Member

@jsnajdr jsnajdr left a comment

Choose a reason for hiding this comment

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

All tests pass and I have nothing to add. Let's :shipit:

@blowery
Copy link
Contributor Author

blowery commented May 25, 2018

I'm gonna sit on this Tuesday unless someone is eager to land it.

Moves us up to the current redux.
@jsnajdr jsnajdr merged commit 729f449 into master May 28, 2018
@jsnajdr jsnajdr deleted the upgrade/redux branch May 28, 2018 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants