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

feat(builtin): add partition function #950

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

lzgabel
Copy link
Contributor

@lzgabel lzgabel commented Nov 11, 2024

Description

  • This PR adds a built in list function partition(list, size).
  • Support for splitting an array into equals sub arrays each with the maximum number of elements defined by the size argument.

Related issues

closes #881

@lzgabel lzgabel requested a review from saig0 as a code owner November 11, 2024 18:23
@saig0
Copy link
Member

saig0 commented Nov 13, 2024

@lzgabel awesome. Thank you for your contribution. 🎉

I'll review your changes in the next few days. Please take a look at the contribution guidelines and check if you followed the guide. 👀

@lzgabel lzgabel force-pushed the feature/881-lz-partition-function branch from 33ff252 to 7d56a15 Compare November 13, 2024 12:47
Copy link
Member

@saig0 saig0 left a comment

Choose a reason for hiding this comment

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

@lzgabel your changes look great. 👍

I added a few comments for minor improvements. Please have a look. 🍪

@lzgabel lzgabel force-pushed the feature/881-lz-partition-function branch from 7d56a15 to 75aa418 Compare November 18, 2024 11:15
@lzgabel
Copy link
Contributor Author

lzgabel commented Nov 18, 2024

Hi @saig0. I have applied your suggestions and squashed the commits. Please check this out. 🙇

@saig0 saig0 self-requested a review November 18, 2024 12:26
Copy link
Member

@saig0 saig0 left a comment

Choose a reason for hiding this comment

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

@lzgabel LGTM. Great work! 🚀


Please follow up and add the new function to the C8 documentation. Refer to the guidelines.

@saig0 saig0 added this pull request to the merge queue Nov 19, 2024
Merged via the queue into camunda:main with commit 18a41b7 Nov 19, 2024
5 checks passed
@lzgabel lzgabel deleted the feature/881-lz-partition-function branch November 19, 2024 15:03
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.

Add partition function that allows to partitioning of collections in an easy way
2 participants