Skip to content

Conversation

@mperry
Copy link
Contributor

@mperry mperry commented Jul 3, 2015

No description provided.

@mperry mperry changed the title Issue162 Fixes #162, fix Validation.sequence and rename to sequenceReduce Jul 3, 2015
@mperry
Copy link
Contributor Author

mperry commented Jul 3, 2015

Fixes #162

@mperry
Copy link
Contributor Author

mperry commented Jul 3, 2015

@tonymorris Is there an accepted name for this? I called it sequenceReduce.

mperry added a commit that referenced this pull request Jul 3, 2015
Fixes #162, fix Validation.sequence and rename to sequenceReduce
@mperry mperry merged commit bd3c4c6 into functionaljava:master Jul 3, 2015
@mperry mperry added the bug label Jul 5, 2015
@mperry mperry added this to the v4.4 milestone Jul 5, 2015
@mperry
Copy link
Contributor Author

mperry commented Jul 5, 2015

I don't know what I was thinking at the time, but I don't like renaming this as sequenceReduce. I suspect I was thinking that reduce is the opposite of accumulate, i.e. the function does not accumulate the error values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant