You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This rule is overly broad by default, preventing things like while (true) with a break inside, which is really common when writing low-level modules like parsers, etc. which is why it was not enabled before now.
However, there is now a checkLoops option that can be set to false so that only things like this get checked:
if(false){// ...}
So I propose we enable this now.
The text was updated successfully, but these errors were encountered:
Ah, sometimes I use this to "trick" standard (if (true) return) with the unreachable code error during development. But overall, this is a good change 👍
Disallow constant expressions in conditions.
http://eslint.org/docs/rules/no-constant-condition
This rule is overly broad by default, preventing things like
while (true)
with abreak
inside, which is really common when writing low-level modules like parsers, etc. which is why it was not enabled before now.However, there is now a
checkLoops
option that can be set tofalse
so that only things like this get checked:So I propose we enable this now.
The text was updated successfully, but these errors were encountered: