Use latest fourmolu #445
Use latest fourmolu #445
Conversation
|
Hm, maybe the formatting changed since that version, and so I need to just change the test? |
|
Also I'm not sure if configs are being loaded correctly, need to investigate. |
|
Thanks for having a go at this! This is very much on my radar. We've just got one more patch to go in before we finally make a major release. I'd expect that'll be on Hackage by the weekend. And I've already rewritten the HLS plugin locally. Perhaps I should have put that here as a draft PR to save you the time. In general, I'll be pretty vigilant about keeping this up to date, since HLS is also the main way I interact with Fourmolu. |
|
Yeah, the test is broken because the default is now to respect import groups. But anyway, this should unfortunately probably be closed in favour of the PR I'll be opening in a few days. |
|
is subsumed by #455 |
Previously PathIdMap cached NormalizedFilePath values, but with the addition of module data those got dropped. It's probably a good idea to bring those back to avoid the risk of a perf regression
My motivation for this is that the latest
fourmoluhas more config options which I want to use.