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 test for left_larger_than_right #1110

Merged
merged 2 commits into from
Jan 28, 2023

Conversation

marijncv
Copy link
Contributor

Signed-off-by: Marijn Valk marijncv@hotmail.com

Description

Adds a test for the left_larger_than_right function and rewrites the function match expression to match on both the left and right argument

Related Issue(s)

Documentation

Signed-off-by: Marijn Valk <marijncv@hotmail.com>
Signed-off-by: Marijn Valk <marijncv@hotmail.com>
Copy link
Collaborator

@wjones127 wjones127 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@wjones127 wjones127 merged commit d984ccf into delta-io:main Jan 28, 2023
@marijncv marijncv deleted the test-left-larger-than-right branch February 5, 2023 09:47
chitralverma pushed a commit to chitralverma/delta-rs that referenced this pull request Mar 17, 2023
Signed-off-by: Marijn Valk <marijncv@hotmail.com>

# Description
Adds a test for the `left_larger_than_right` function and rewrites the
function match expression to match on both the `left` and `right`
argument

# Related Issue(s)
<!---
For example:

- closes delta-io#106
--->

# Documentation

<!---
Share links to useful documentation
--->

---------

Signed-off-by: Marijn Valk <marijncv@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate rust
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants