Skip to content

Conversation

@daveisfera
Copy link
Contributor

Fixes #35

@daveisfera
Copy link
Contributor Author

daveisfera commented Sep 29, 2020

Also, fixed the handling for of 0 in the path because it was being rejected for being falsy

@daveisfera
Copy link
Contributor Author

I added code to fix #24 as well

@villesau
Copy link
Owner

Thanks for the PR @daveisfera ! And sorry for super late reply.

Looks like the tests are failing on this one. I've added github actions to the repo so by rebasing you'll see the results here too.

@daveisfera
Copy link
Contributor Author

I no longer have access to the codebase that I wrote this for, so I don't have a way to test/change anything with this PR. if you don't have the time/interest to fix it, then go ahead and close it.

@MasonChinkin
Copy link

@villesau I'm gonna take a shot at getting this PR passing. Wish me luck. Thanks for making this codemod!

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.

Doesn't handle getOr from lodash/fp

3 participants