search

AmpersandJS / amp

a collection of individual JS utility modules
http://amp.ampersandjs.com
MIT License
130 stars 12 forks source link

Add union and intersection. #76

Closed latentflip closed 9 years ago

latentflip commented 9 years ago

Basically just a straight port from underscore. Required for AmpersandJS/ampersand-state#131

Closes #75

wraithgar commented 9 years ago

:+1:

fyockm commented 9 years ago

Interesting to me that intersection has a couple tests around null, and union has tests for nested arrays - but not vice versa. Might be nice to add those. Otherwise LGTM.

latentflip commented 9 years ago

@fyockm that was just straight from underscore tests. Will add some more.

HenrikJoreteg commented 9 years ago

@wraithgar suggested we can add additional tests post 1.0.0 on this, given that it's a straight port from underscore.

HenrikJoreteg commented 9 years ago

+1 from me, will ship once CI passes.

HenrikJoreteg commented 9 years ago

published.