search

avh4 / elm-upgrade

Upgrade Elm projects
https://www.npmjs.com/package/elm-upgrade
MIT License
122 stars 15 forks source link

Missing parens when inserting Range #8

Closed manuscrypt closed 7 years ago

manuscrypt commented 7 years ago

@av: I just tried to elm-upgrade a project (elm-color-extra, obvisouly not mine), and noticed this diff:

-            List.map2 (\i cl -> ( (toFloat i / toFloat l), cl )) [0..l] palette
+            List.map2 (\i cl -> ( (toFloat i / toFloat l), cl )) List.range 0 l palette

And the compiler complains about the number of arguments to map2.

avh4 commented 7 years ago

Ah, this bug should go to -> https://github.com/avh4/elm-format/issues

avh4 commented 7 years ago

Fixed in https://github.com/avh4/elm-format/issues/271 will be in elm-format 0.5.1-alpha