Hey all,
I released a new few versions for elm-review and its packages over the last hour and month.
jfmengels/elm-review (2.7.X)
Thanks to Jie, there is now a new Rule.filterErrorsForFiles function to precisely filter the files you wish to get errors for. This can be useful if you wish to enable a rule only in some files that match a pattern (all Example.elm files for instance). Jie used this function to integrate elm-review into Exercism’s Elm track to make detailed feedback for user’s solutions, which is super cool ![]()
It also contains also a smaller issue to fix elm-review --watch not always updating when files get added or deleted.
elm-review-unused (1.1.21)
- Fixed some false positives in NoUnused.Exports (thanks @janiczek for all the help, Don't report exposed custom types whose constructors are used elsewhere by jfmengels · Pull Request #62 · jfmengels/elm-review-unused · GitHub)
- If you didn’t know,
elm-reviewsupports Lamdera (my own rules do at least), but there was an issue wheremainwas reported as unused in Lamdera applications (Module reported as unused when it contains main · Issue #54 · jfmengels/elm-review-unused · GitHub) NoUnused.Variablesnow no longer removes let_declarations if the assigned value is a call toDebug.log. This is to prevent people from doing ugly hacks to keepDebug.logs around in files where those are okay to keep (examples/styleguides for instance). If you still wish to remove those, enablingNoDebug.Logwill make sure everything gets cleaned up
elm-review-simplify (2.0.8)
Now also simplifies these use-cases:
String.replace x y ""
--> ""
String.replace x x z
--> z
String.replace "x" "y" "z"
--> "z" -- only when resulting string is unchanged
Json.Decode.oneOf [ a ]
--> a
Parser.oneOf [ a ]
--> a
Hope you find these helpful ![]()
