Skip to content

Added method to do ZIP with array of arrays.#1

Merged
isaacroldan merged 3 commits into
developmentfrom
feature/zipAll
Feb 23, 2015
Merged

Added method to do ZIP with array of arrays.#1
isaacroldan merged 3 commits into
developmentfrom
feature/zipAll

Conversation

@isaacroldan

Copy link
Copy Markdown

🎩 What? Why?

There is already a zipWith method but i think this one is more useful.
Zip an array of arrays.
this:

@[@[@1, @2, @3], @[@4, @5, @6], @[@7, @8, @9]]

transforms into this:

@[@[@1, @4, @7], @[@2, @5, @9], @[@3, @6, @9]]

⚠️ all the arrays should have the same lenght

👻 GIF

@isaacroldan

Copy link
Copy Markdown
Author

@masylum @sergigracia

@mariabernis

Copy link
Copy Markdown

👍

@masylum

masylum commented Feb 20, 2015

Copy link
Copy Markdown

I don't see tests nor a PR to the original repo!

@isaacroldan

Copy link
Copy Markdown
Author

The PR to the original Repo is this one:
robb#60

i'm working in tests, but they use a very old cocoapods version and i can't run them :(

Comment thread Underscore/USArrayWrapper.m Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

typo s/lenght/length

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

and 🎨 spaces between operators

@isaacroldan

Copy link
Copy Markdown
Author

I'll add tests in another moment, I can't even compile their test project.

Comment thread Underscore/USArrayWrapper.m Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

typo is still here....

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

agh, stupid me, sorry, i'll fix it

@sergigracia

Copy link
Copy Markdown

👍

isaacroldan added a commit that referenced this pull request Feb 23, 2015
Added method to do ZIP with array of arrays.
@isaacroldan isaacroldan merged commit 9b5a534 into development Feb 23, 2015
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.

4 participants