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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(mapStateToProps): port to typescript #1745

Merged

Conversation

tony-go
Copy link
Contributor

@tony-go tony-go commented Jun 29, 2021

Hi folks 馃憢

鈿狅笍 : Lot of any there. I have to make a second pass to add generics. But feel free to give me feedback anyway.

This pull request follows the typescript migration initiative (#1737).

files to migrate:

  • mapStateToProps
  • wrapMapToProps

@netlify
Copy link

netlify bot commented Jun 29, 2021

鉁旓笍 Deploy Preview for react-redux-docs ready!

馃敤 Explore the source changes: 4137320

馃攳 Inspect the deploy log: https://app.netlify.com/sites/react-redux-docs/deploys/60de22779baea60007354dd4

馃槑 Browse the preview: https://deploy-preview-1745--react-redux-docs.netlify.app

@tony-go tony-go changed the title chore(mapStateToProps): move to ts first pass chore(mapStateToProps): port to typescript Jun 29, 2021
@markerikson
Copy link
Contributor

Why did package-lock.json change in this PR, and so massively? That doesn't seem right.

src/connect/wrapMapToProps.ts Outdated Show resolved Hide resolved
src/connect/wrapMapToProps.ts Outdated Show resolved Hide resolved
@timdorr timdorr force-pushed the typescript-port-mapStateToProps branch from 366e3cd to 9c7ca0e Compare June 29, 2021 20:47
@timdorr
Copy link
Member

timdorr commented Jun 29, 2021

Why did package-lock.json change in this PR, and so massively? That doesn't seem right.

Looks like the lockfile on master was updated to v2, but not on the TS branch. I just fixed that up now.

@tony-go tony-go force-pushed the typescript-port-mapStateToProps branch from 0fad9ff to 4137320 Compare July 1, 2021 20:15
@tony-go
Copy link
Contributor Author

tony-go commented Jul 1, 2021

Not sure, but feel like I'm reaching something.

Let me know @markerikson .

@markerikson markerikson marked this pull request as ready for review July 2, 2021 01:26
@markerikson
Copy link
Contributor

Ought to be good for now. Thanks for working on this!

@markerikson markerikson merged commit 0cece74 into reduxjs:typescript-port Jul 2, 2021
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.

None yet

3 participants