fsprojects / FSharp.Control.Reactive

Extensions and wrappers for using Reactive Extensions (Rx) with F#.
http://fsprojects.github.io/FSharp.Control.Reactive
Other
284 stars 60 forks source link

Breaking Change: switch order of parameters #21

Closed panesofglass closed 10 years ago

panesofglass commented 10 years ago

Most F# operators that take two of a kind, e.g. List.zip, take item1 then item2. In FSharp.Reactive, we flipped these to better support use of the pipeline operator |>.

Should we flip these back? Perhaps with the next update to Rx 2.4?

@dsyme, @forki, thoughts?

panesofglass commented 10 years ago

Fixed in commit 19467490ed0be573761e381da63b6bf4dabcb7ce