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

Split tests up #266

Closed jfmengels closed 12 months ago

jfmengels commented 12 months ago

The SimplifyTest file has just reached an impressive 30 000 lines of code. And my computer fan is the one telling me it's impressive, making it quite hard/slow to work with (even elm-format is really struggling to work with it reliably for me).

I've been thinking about splitting it up, roughly by the existing describes we have in that file (so one file for Dict, one file for Set, etc.).

Does that sound like an okay plan to you, or would this be more of a pain than a help?

lue-bird commented 12 months ago

sounds good to me