Glavin001/tslint-clean-code

New rule: avoid-if-else

Open

#39 ouverte le 28 janv. 2018

Voir sur GitHub
 (3 commentaires) (1 réaction) (0 assignés)TypeScript (15 forks)github user discovery
good first issuerule

Métriques du dépôt

Stars
 (175 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

http://blog.timoxley.com/post/47041269194/avoid-else-return-early

Use of if-else is always unnecessary, and leads to cyclomatic complexity. Other clean code linters (phpmd, golint) enforce this pattern. Would love to have this rule as I have to enforce it manually in my typescript projects right now.

Guide contributeur

Direction de recherche
Étudiez les règles 'no else return' existantes dans ESLint et d'autres linters, puis implémentez une règle similaire dans TSLint en suivant son API de règles.
Stack technique
typescript
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptTSLint rule development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60