Rule request: no-useless-return

Please describe what the rule should do:

Forbid return; at the end of a function, as it is redundant. It’s often a leftover from refactoring some callback code.

What category of rule is this? (place an “X” next to just one item)

[x] Enforces code style [ ] Warns about a potential error [ ] Suggests an alternate way of doing something [ ] Other (please specify:)

Provide 2-3 code examples that this rule will warn about:

The common case:

function foo() {
  bar();
  return; // pointless!
}

Generalising to conditionals at the end of a function:

function foo() {
  if (something) {
    bar();
    return; // redundant
  } else if (otherThing) {
    baz();
    return; // redundant
  } else {
    boz();
  }
  // Nothing here, so the arms of the conditional are the last thing
  // in the function.
}

Examples where we wouldn’t warn:

function foo() {
  bar();
  return 1;
}

Why should this rule be included in ESLint (instead of a plugin)?

It’s totally generic and applicable to all JS developers I think.

[mysticatea]

I also can implement this since I like code path analysis. 😄

[platinumazure]

Sounds great! I’ll endorse this. With two more endorsements and a champion, we can accept this.