Add of support for 'unique' method on array wrapper

[akinsella]

Hi,

I added some support for 'unique' method on array wrapper.

Here is a sample:

        NSURL *fbReposUrl = [NSURL URLWithString:@"https://api.github.com/orgs/facebook/repos"];
        NSData *data = [NSData dataWithContentsOfURL:fbReposUrl];
        NSArray *repos = [NSJSONSerialization JSONObjectWithData:data options:kNilOptions error:NULL];

        NSArray *languages = _array(repos).pluck(@"language").reject([Underscore isNull]).unique.unwrap;

        NSLog(@"Unique languages used by FaceBook: %@", languages);

[robb]

This block in not necessary.
I guess you based this on the flatten implementation but there it's just for recursion :-)

[akinsella]

Hi,

My mistake. Yes, I based the change on flatten implementation. Do I need to resubmit the pull request ?

[robb]

While you can just push to this branch to update the PR, I don't think it makes sense to target development from here.

I suggest you reimplement it based on development and submit a new PR. Sorry for the trouble.

[robb]

Hi Alexis.

Would you mind addressing my comment and create a new pull request against the development branch?

That branch contains a couple of syntax changes, most notably it adds a functional style for calling methods:

NSArray *myArray = @[@1, @2, @2, @3, @1];
NSArray *uniques = Underscore.unique(myArray);

I'm a bit behind with pushing version 0.2, sorry for that!

Thanks!

[akinsella]

Ok,
I'll do it right now.

[robb]

Cool, I close this Pull Request then

[robb]

Also, let's call it uniq to be in sync with Haskell, Underscore.js and Unix :-)