search

tomjkidd / elm-multiway-tree-zipper

A library for navigating and updating immutable trees.
29 stars 5 forks source link

Documentation code snippets incompatible with Elm 0.18.0 in version 1.10.1 #16

Closed inactivist closed 8 years ago

inactivist commented 8 years ago

The &> infix operator as defined doesn't work as expected because Maybe.andThen params are reversed in 0.18.0.

(Haven't verified this. Expectation based on experience upgrading this module to Elm 0.18.0.)

tomjkidd commented 8 years ago

I will have to devote some time to this in the coming days, the 0.18.0 upgrade is still new to me and I will have to look around for if there is a new standard way of dealing with chaining.

tomjkidd commented 8 years ago

Turns out this was really easy to fix, just have to use (&>) = flip Maybe.andThen

inactivist commented 8 years ago

Thanks for investigating and fixing this - I admit I was unaware of flip (new to Elm -- though it's pretty obvious now that I think about it).