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

CU-fv6xq2 Clean up Arrow Syntax based on API Review #2328

Merged
merged 2 commits into from Mar 18, 2021

Conversation

nomisRev
Copy link
Member

This PR cleans up the API for Arrow Syntax for Arrow 0.13.0 (1.0.0). The API review can be found here

  • Reduce to arity-5 instead of 22
  • Remove arity post-fix from Partials.kt
  • Deprecate Complement
  • Deprecate pipe
  • Deprecate reverse
  • Deprecate tupling.kt
  • Deprecate pairing.kt

@nomisRev nomisRev requested a review from a team March 17, 2021 13:33
@franciscodr
Copy link
Collaborator

Copy link
Member

@raulraja raulraja left a comment

Choose a reason for hiding this comment

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

thanks @nomisRev !

@raulraja raulraja requested a review from a team March 17, 2021 13:36
@nomisRev nomisRev changed the title Apply API review comments CU-fv6xq2 Clean up Arrow Syntax based on API Review Mar 17, 2021
@i-walker i-walker merged commit 254a995 into master Mar 18, 2021
@rachelcarmena rachelcarmena deleted the cu-fv6xq2-clean-up-arrow-syntax-review branch March 19, 2021 09:59
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

4 participants