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

fix warning in replaceReducer test #3527

Merged

Conversation

cellog
Copy link
Contributor

@cellog cellog commented Aug 29, 2019

The replaceReducer new reducer in the test was only returning 5, which is almost certainly an error in the real world. This PR changes the test so that it returns a more realistic object, removing the warning

@timdorr timdorr merged commit 4f7ff64 into reduxjs:ts-conversion Aug 29, 2019
@cellog cellog deleted the cellog-3482-replaceReducer-test branch August 29, 2019 14:56
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
Former-commit-id: 3bfb592
Former-commit-id: e3f9dc3
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
Former-commit-id: 3bfb592
Former-commit-id: e3f9dc3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants