sindresorhus/eslint-plugin-unicorn
Rule proposal: `no-useless-else`
Fechada
#987 aberto em 2 de jan. de 2021
help wantednew rule
Métricas do repositório
- Stars
- (5.022 estrelas)
- Métricas de merge de PR
- (Mesclagem média 1d 16h) (399 fundiu PRs em 30d)
Description
If the body of if contains one of these keywords, the else is moot:
throwbreakcontinuereturn
This slightly overlaps with https://eslint.org/docs/rules/no-else-return
This was inspired by https://github.com/eslint/eslint/issues/13067.
This could be auto-fixed, but we should take care to properly move any comments.
Fail
if (foo) {
throw new Error();
} else {
console.log('🦄');
}
Pass
if (foo) {
throw new Error();
}
console.log('🦄');