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.

Sign up for GitHub

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

Jump to bottom

Make async.transform actually support 2 arguments #1381

Merged
merged 3 commits into from Mar 14, 2017
Merged

Make async.transform actually support 2 arguments #1381

merged 3 commits into from Mar 14, 2017

Commits on Mar 13, 2017

  1. Make async.transform actually support 2 arguments 

    The documentation suggests that both the `accumulator` and `callback` arguments to transform is optional. 

However, if both are left out (and transform therefore is called with only 2 arguments), transform crashes with a `iteratee is not a function` exception. 
This fixes that.
    @erik-krogh
    erik-krogh committed Mar 13, 2017
    Copy the full SHA
    aa3fbfe View commit details
    Browse the repository at this point in the history

  2. Added test for calling transform with two arguments

    @erik-krogh
    erik-krogh committed Mar 13, 2017
    Copy the full SHA
    d7bc238 View commit details
    Browse the repository at this point in the history

Commits on Mar 14, 2017

  1. Added test for filter with an object-collection, where the filtering … 

    …fails. To improve coverage.
    @erik-krogh
    erik-krogh committed Mar 14, 2017
    Copy the full SHA
    3e8377d View commit details
    Browse the repository at this point in the history