search

clj-commons / manifold

A compatibility layer for event-driven abstractions
1.02k stars 106 forks source link

deferred: add zipseq and zipseq' #170

Closed pyr closed 3 years ago

pyr commented 5 years ago

Hi!

In a number of cases I find myself having to wait on a collection of deferreds. While it's fine to apply zip', I find this to improve legibility of the resulting code.

Am I missing an obvious way to do this?

pyr commented 5 years ago

(the build failure seems unrelated and triggers on master as well)

pyr commented 5 years ago

If #172 goes in I will rebase on top of it to make the build pass

pyr commented 5 years ago

It now builds thanks to #172, happy to have more thoughts on whether this is a good idea or not

lurodrigo commented 4 years ago

I think that's a great idea, I also need to (apply d/zip' deferreds) often.

pyr commented 3 years ago

any interest in this one?

KingMob commented 3 years ago

@pyr Thanks again for your hard work, but I really think the new functions are just too similar. While I agree zipseq's param behavior may be the better default choice compared to when Manifold was first written, I'm not sure the improvement is enough to justify having two sets of almost-identical fns in the namespace.