-
Notifications
You must be signed in to change notification settings - Fork 37
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: pass join_nulls
and validate
to the lazy method for $join()
#949
Conversation
join_nulls
and validate
to the lazy method for $join()$
join_nulls
and validate
to the lazy method for $join()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @george-wood, that looks good to me. Can you also add tests for the arg
validate
for DataFrame
?
I think the CI failure is unrelated
Add tests for the `validate` argument for DataFrame joins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you resolve conflicts of the NEWS file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the contribution!
Co-authored-by: eitsupi <50911393+eitsupi@users.noreply.github.com>
Thanks for reviewing! |
Fix #945.