Skip to content

add overloads to compose.ts and applyMiddleware.ts #3558

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 2, 2019
Merged

add overloads to compose.ts and applyMiddleware.ts #3558

merged 2 commits into from
Sep 2, 2019

Conversation

cellog
Copy link
Contributor

@cellog cellog commented Sep 2, 2019

Because applyMiddleware relies upon compose, these two need to be committed at the same time. The comment changes in compose.ts are because of subtle differences between the comments in index.d.ts and compose.ts. Where there was a conflict I chose the index.d.ts version

@timdorr timdorr merged commit 8112c95 into reduxjs:ts-conversion Sep 2, 2019
@reduxjs reduxjs deleted a comment from kevinaskin Nov 4, 2019
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
@cellog @timdorr
add overloads to compose.ts and applyMiddleware.ts (reduxjs#3558)
7b39aba 
* add overloads to compose.ts

* add applyMiddleware overload fixes


Former-commit-id: 6c609ac
Former-commit-id: 3774004
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
@cellog @timdorr
add overloads to compose.ts and applyMiddleware.ts (reduxjs#3558)
58c9dc4 
* add overloads to compose.ts

* add applyMiddleware overload fixes


Former-commit-id: 6c609ac
Former-commit-id: 3774004
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cellog @timdorr