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

Add support for getOr #36

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

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