Teleport imports #512

[perkee]

Solve issue #512. 100% AI written and I 100% understand if this goes nowhere. Making this binary my main driver for the next little while and if I like it, I'll take this out of draft.

• First commit is getting nix to work on my machine. Probably I'm holding it wrong and this should go Dropped because I had not rebased in far too long. Sorry
• first commit is the feature, including the plan, using this prompt:

   let's add a feature to elm-format whereby an import statement can appear on any line in a module and it will be moved to the import block. The best way to do this is to add a test wherein some imports appear
  after the top level declarations have started in the file. The test should assert that the output of running elm-format on such a file produces an elm file that compiles with the elm compiler. Create a plan in
  a markdown file including these steps and any intermediate steps, then iterate on the solution and check off the checklist in the markdown file.

• second commit the plan is deleted and I create a homebrew formula to allow brew install perkee/elm/elm-format and a nix flake that never really works right.
• Third commit delete everything created in the second commit

[kutyel]

@avh4 I think this is a nice addition indeed and I don't see any particular issue with the code, however, IMHO I think the Nix and homebrew/CI changes should be split to a separate PR 🙏🏻

[perkee]

@kutyel Hi Flavio!!

I think the Nix and homebrew/CI changes should be split to a separate PR

They should be dropped entirely! Sorry, they only exist to make it easier to test a binary this fork of elm-format.

[perkee]

@kutyel yep dropped. Taking this out of draft because my playtesting has it running like a Swiss watch.

[kutyel]

LGTM!

[kutyel]

LGTM

[perkee]

What's the story with these CI checks? They've been "Waiting for status to be reported" for days now.

[lydell]

GitHub Actions runs must be approved by a maintainer when the PR is from a new contributor, to avoid people running crypto miners and malicious stuff on random repos (remember, you can of course change the workflow yaml files in the PR).

[perkee]

ohhhh good call @lydell! TYSM

[perkee]

@avh4 hi