block-scoped-var and no-redeclare are probably mostly due to multiple for loops using i in the same function scope. Most likely this can be simply solved by using the much cleaner forEach.
max-depth only had one instance as far as I can remember, but it was pretty deeply nested code that I'd rather refactor in its own commit for obvious reasons.
eqeqeq is mostly just figuring out what the hell is a string and what's not. It'll probably help make typing just a bit more clear hopefully.
Hopefully we'll be able to go rule by rule and remove them from the list of stuffed errors.
Excerpt from
app/assets/javascript/.eslintrc.json
block-scoped-var
andno-redeclare
are probably mostly due to multiple for loops usingi
in the same function scope. Most likely this can be simply solved by using the much cleanerforEach
.max-depth
only had one instance as far as I can remember, but it was pretty deeply nested code that I'd rather refactor in its own commit for obvious reasons.eqeqeq
is mostly just figuring out what the hell is a string and what's not. It'll probably help make typing just a bit more clear hopefully.Hopefully we'll be able to go rule by rule and remove them from the list of stuffed errors.