search

tfausak / flow

:droplet: Write more understandable Haskell.
https://hackage.haskell.org/package/flow
MIT License
198 stars 10 forks source link

Allow mixing operators #4

Closed tfausak closed 9 years ago

tfausak commented 9 years ago

Fixes #3.

tfausak commented 9 years ago

One problem with this approach is that <| doesn't behave like you might expect.

f x y z
x |> f <| y <| z -- Wrong!

Since <| is meant to represent right-to-left pipelines, it doesn't work well with multiple arguments.

h (g (f x))
h <| g <| f <| x
tfausak commented 9 years ago

@tonyday567 are you interested in this pull request?

tonyday567 commented 9 years ago

Hi tf, I see what you mean, but I don't work well with multiple arguments either ;)

I'd say that this precedence is marginally better, but don't consider my opinion well-thought.

tfausak commented 9 years ago

I agree that it's marginally better. I'm not sure if it's worth adding this or not.

tonyday567 commented 9 years ago

Leave it out then, IMO. I think the best path for this library is to maintain design clarity, and be ruthless about scope creep.

tfausak commented 9 years ago

:+1: I think that's a good goal for any library.