feat: disable curly rule

[stefvw93]

Disable curly

• current setup disallows single line control flow; subjective, but annoying imo, I want to if (statement) return

[thomasowow]

JavaScript allows the omission of curly braces when a block contains only one statement. However, it is considered by many to be best practice to never omit curly braces around blocks, even when they are optional, because it can lead to bugs and reduces code clarity.

Kind of agree with this, mostly for consistency. Inconsistency sometimes is less readable in my opinion.

[stefvw93]

@thomasowow @bb441db I added multi-or-nest

Incorrect

/*eslint curly: ["error", "multi-or-nest"]*/

if (!foo)
    foo = {
        bar: baz,
        qux: foo
    };

while (true)
  if(foo)
      doSomething();
  else
      doSomethingElse();

if (foo) {
    foo++;
}

while (true) {
    doSomething();
}

for (var i = 0; foo; i++) {
    doSomething();
}

correct

/*eslint curly: ["error", "multi-or-nest"]*/

if (!foo) {
    foo = {
        bar: baz,
        qux: foo
    };
}

while (true) {
  if (foo) doSomething();
  else doSomethingElse();
}

if (foo) foo++;

while (true) doSomething();

for (var i = 0; foo; i++) doSomething();

[barisbora]

Just question it covers return?

[barisbora]

I think we should keep return at least, I also really like if and for too

[stefvw93]

Agree, though: Could you create a separate PR for this? 🙏🏻