search

jfmengels / elm-review-simplify

Provides elm-review rules to simplify your Elm code
https://package.elm-lang.org/packages/jfmengels/elm-review-simplify/latest/
BSD 3-Clause "New" or "Revised" License
19 stars 9 forks source link

Compute import lookup once after imports have been visited #89

Closed jfmengels closed 1 year ago

jfmengels commented 1 year ago

I think we can get rid of the "maybe cache". It is possible that to support the let variables, we may need to touch that import lookup when switching scopes in the project and need to recompute the import lookup somewhat, but we should avoid to recompute the things I moved to afterImportsVisitor anyway.