add catOptions and equalsWith functions for seq, list, array and map#64
Conversation
There was a problem hiding this comment.
where is the difference to Seq.choose id?
There was a problem hiding this comment.
none, the point is to have one function that does this, so that you don't have to compose two other to do the job (like in Haskell catMaybes)
There was a problem hiding this comment.
(Seq.choose id) is also one function and can be composed!?
There was a problem hiding this comment.
yes, but that's already two variables that have to have parentheses around them -- catOptions is supposed to be only a small helper function, I wrote it because I've found myself writing (Seq.choose id) way to often. DRY
There was a problem hiding this comment.
I can rewrite them in terms of (choose id) if you think that would be better
There was a problem hiding this comment.
mmm tbh I don't really agree gere. But maybe we find others that can join discussion
sideeffffect
changed the title
|
I've implemented the @forki could you also please have a look at fsprojects/ProjectScaffold#270 and fsprojects/FSharpx.Async#31 ? others seem to be a bit passive. Thank you very much! |
…o Seq, List and Array
sideeffffect
force-pushed
the
master
branch
from
b49ca85 to
0cc2d4b
Compare
|
@tpetricek could you please have a look at this? |
|
@forki @mausch @panesofglass @tpetricek |
2 participants
No description provided.