pine-vm / pine

Cross-platform runtime environment for the Elm programming language.
https://pine-vm.org
MIT License
217 stars 17 forks source link

Simplify Elm app code: Automate add the required import statements #2

Closed Viir closed 4 years ago

Viir commented 4 years ago

If imports are missing in the ElmFullstackLoweringInterface.GenerateJsonCoders, that is easier to see in the lowered app code. It would be nicer if these are added automatically, analogous to the compilation of the backend.

https://github.com/elm-fullstack/elm-fullstack/blob/b95bd3bb2daa44a4fce02dce3b98cff4762ecf9a/implement/PersistentProcess/PersistentProcess.Common/ElmApp.cs#L209-L458

Viir commented 4 years ago

Done with commit https://github.com/elm-fullstack/elm-fullstack/commit/a86d098bd1f8e6d0cdbf652825107faddd341e1d