mdgriffith / elm-codegen

https://package.elm-lang.org/packages/mdgriffith/elm-codegen/latest/
BSD 3-Clause "New" or "Revised" License
138 stars 16 forks source link

fix imports within letin expressions #22

Closed kirchner closed 2 years ago

kirchner commented 2 years ago

Hi,

it seems like, that imports are not propagated outside of let in expressions. The change seems to fix this and also still works with my personal test project. I also added a test.

Best, Fabian

mdgriffith commented 2 years ago

Thank you and good catch! I appreciate the inclusion of a test as well!

I ended up removing the top level Elm.letIn in favor of the Elm.Let module which should be easier to use.

kirchner commented 2 years ago

Sounds good :)