This repository has been archived by the owner on Jul 25, 2018. It is now read-only.

Should concat be associative? #2

Open

jimf opened this issue Apr 10, 2016 · 0 comments

jimf commented Apr 10, 2016

> Validation.Failure(['message1']).concat(Validation.Success('foo')).concat(Validation.Failure(['message2']))
definitions { f: [ 'message2' ] }

> Validation.Failure(['message1']).concat((Validation.Success('foo')).concat(Validation.Failure(['message2'])))
definitions { f: [ 'message1', 'message2' ] }

Just wanna make sure this is intentional. I was expecting both of these to return the aggregated failures.

The text was updated successfully, but these errors were encountered:

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