Open jfmengels opened 2 months ago
I had a go through the codebase, and found a bunch of performance optimizations that could be applied.
(The only one with a perf trade-off is I think the last one, let me know if I should keep or remove it)
Also upgrades elm-review and adds Simplify and NoPrematureLetComputation, which found a few issues.
elm-review
Simplify
NoPrematureLetComputation
I wish you a happy reviewing of an unrequested large PR :stuck_out_tongue:
I had a go through the codebase, and found a bunch of performance optimizations that could be applied.
(The only one with a perf trade-off is I think the last one, let me know if I should keep or remove it)
Also upgrades
elm-reviewand addsSimplifyandNoPrematureLetComputation, which found a few issues.I wish you a happy reviewing of an unrequested large PR :stuck_out_tongue: