Rule request: forbid return in finally

[louy]

Adding a return statement in a finally block is a very confusing thing. Thrown errors get ignored, return statements in try blocks get ignored, and your mind explodes.

I'd love to have a rule to forbid this. Especially now that we're having async/await.

examples:

try {
return 1;
} catch(err) {
return 2;
} finally {
return 3;
}
// > 3

try {
throw new Error;
return 1;
} catch(err) {
return 2;
} finally {
return 3;
}
// > 3

[adidahiya]

sounds reasonable. is there any analogous rule for ESLint? if not, that's cool too

[louy]

not yet. eslint/eslint#5808

[andy-hanson]

Should be fixed by #1349.