Cannot be used with Bogdanp/elm-combine

Question

Cannot be used with Bogdanp/elm-combine

roSievers opened this issue 8 years ago · comments

Your new infix operator precedences clash because of a compiler bug. I would love to use both of your libraries, can you find a way to sort that out?

Bug at elm-combine: https://github.com/Bogdanp/elm-combine/issues/19
Compiler bug: elm/compiler#1096
This isn't the first time that happens to Bogdan: https://github.com/Bogdanp/elm-combine/issues/18

For anyone else with the same problem, ditching this for Bloom's fork solves the issue, because they are behind on the infix operators: http://package.elm-lang.org/packages/bloom/elm-return/latest

Isaac Shapira commented 8 years ago

Ok

Isaac Shapira · Answer 1 · Wed Nov 23 2016 02:18:46 GMT+0800 (China Standard Time)

I recently merged a PR from @splodingsocks is this still an issue?

Rolf Kreibaum · Answer 2 · Wed Nov 23 2016 03:42:54 GMT+0800 (China Standard Time)

The last commit was 7 days ago (except for the license) and I stumbled on the problem six days ago. The Problem are the infixl definitions and they are still in your code. (Also, that merge does not show up for me or github doesn't know it was a merge.)

I believe the problem can't be solved without removing infixl declaration from either package and thus removing some functionality. That's unfortunate and I'm not saying either of you has to do that. (Especially since I have a workaround)

Isaac Shapira · Answer 3 · Wed Nov 23 2016 07:08:20 GMT+0800 (China Standard Time)

Here ya go. All fixed up 6bd87ec

Rolf Kreibaum · Answer 4 · Wed Nov 23 2016 16:23:13 GMT+0800 (China Standard Time)

Thanks, I just tested compatibility by switching back to your version of the package. It works great.