search

unisonweb / unison

A friendly programming language from the future
https://unison-lang.org
Other
5.78k stars 270 forks source link

a PR where we move `ORMOLU_DISABLE` to below the module imports #2685

Closed aryairani closed 2 years ago

aryairani commented 2 years ago

Follow-on to #2666.

HLS puts imports in the wrong spot, and rather than continuing to try to thread them in manually without disturbing the surrounding imports, or not (and increasing the entropy), let's do this.

ChrisPenner commented 2 years ago

I can probably script something to make this patch, but of course in the mean-time feel free to just move it down as it causes issues.

This'll also be nice for avoiding conflicts on imports I think, since HLS always adds them at the end which causes contention, whereas ormolu sorts them.

aryairani commented 2 years ago

image Apparently the comment also needs to be moved to a separate line.

aryairani commented 2 years ago

Let's wait until after #2471

ChrisPenner commented 2 years ago

Closed in #2956