Fourmolu fixup

[kostmo]

Switch to fourmolu-0.13 and reformat all source code.

Closes #1328.

[byorgey]

I think Haddock chokes on this. But once we rebase this PR on top of #1308 it should be fine.

[byorgey]

I tried updating to fourmolu-0.13.0.0 but sadly it seems to make a hash of some of our parser code. It seems like it now has the wrong idea about the precedence of <|> vs other operators. For example:

-        string "0b" *> L.binary
-          <|> string "0o" *> L.octal
-          <|> string "0x" *> L.hexadecimal
-          <|> L.decimal
+        string "0b"
+          *> L.binary
+            <|> string "0o"
+          *> L.octal
+            <|> string "0x"
+          *> L.hexadecimal

[byorgey]

OK, got it figured out. By reading this: fourmolu/fourmolu#345 I figured out that we just need to give fourmolu a few hints about modules that re-export other modules, so that it knows where certain operators come from (and hence what their precedence is).

[byorgey]

@kostmo what do you think? I think this is ready to go. Once we rebase on top of #1367 it should build cleanly, I think.

[kostmo]

@kostmo what do you think? I think this is ready to go. Once we rebase on top of #1367 it should build cleanly, I think.

Yes, just need your approval to merge.

[byorgey]

Yes, just need your approval to merge.

Sure, since I made some changes too I wanted you to take a look. I guess I could have just approved it and then let you merge it. 😄