no-arrow-condition: disallow arrow functions where test conditions are expected

This rule was removed in ESLint v2.0 and replaced by a combination of the no-confusing-arrow and no-constant-condition rules.

Arrow functions (=>) are similar in syntax to some comparison operators (>, <, <=, and >=). This rule warns against using the arrow function syntax in places where a condition is expected. Even if the arguments of the arrow function are wrapped with parens, this rule still warns about it.

Here's an example where the usage of => is most likely a typo:

// This is probably a typo
if (a => 1) {}
// And should instead be
if (a >= 1) {}

There are also cases where the usage of => can be ambiguous and should be rewritten to more clearly show the author's intent:

// The intent is not clear
var x = a => 1 ? 2 : 3
// Did the author mean this
var x = function (a) { return a >= 1 ? 2 : 3 }
// Or this
var x = a <= 1 ? 2 : 3

Rule Details

Examples of incorrect code for this rule:

/*eslint no-arrow-condition: "error"*/
/*eslint-env es6*/

if (a => 1) {}
while (a => 1) {}
for (var a = 1; a => 10; a++) {}
a => 1 ? 2 : 3
(a => 1) ? 2 : 3
var x = a => 1 ? 2 : 3
var x = (a) => 1 ? 2 : 3

Related Rules

Version

This rule was introduced in ESLint 1.8.0 and removed in 2.0.0-beta.3.

Resources