Re:Disable comments make static analysis tools worse
by jfmengels1 on Tuesday July 06, 2021 @08:53AM
(#27747834)
Just to clarify: Since elm-review is a static analysis tool separate from the compiler, you can run and compile your program just fine without having to handle all the dead code warnings (which elm-review is actually very good at detecting https://jfmengels.net/safe-dead-code-removal/).
I myself personally tend to run elm-review when I'm ready to make a PR most of the times.
Imo, this misses an important use case for warnings: reporting issues that I don't want in committed code, but that are fine while developing. For example, dead code — I don't want to commit a function that's never called, but I do want to be able to compile/run my code and run any tools over it when I've written a new function but not yet called it.