Use exposingList utility function from Elm.Syntax.Module

mbuscemi / elm-lens

Elm code visualizations for maximum productivity in Atom

https://atom.io/packages/elm-lens

MIT License

51 stars 2 forks source link

Use exposingList utility function from Elm.Syntax.Module #13

Closed stil4m closed 6 years ago

stil4m commented 6 years ago

Less is more :)

mbuscemi commented 6 years ago

Funny how I missed that. I was just looking at that this morning and thinking it could be cleaner. Thanks!

stil4m commented 6 years ago

I'm looking to move some functions from elm-analyse into elm-syntax. Turned out I had that function duplicated over the two projects :/.

mbuscemi commented 6 years ago

Please let me know if there are any other ways I can optimize my usage of elm-syntax. Also, if any existing Elm Lens code could be turned into more generic behavior for elm-syntax, I'm open to doing those refactors.

stil4m commented 6 years ago

Yes I will. I'm preparing a PR for this, but I also require a major version bump on elm-analyse. I want to be sure that I get the most out of that major change.