airbnb / airbnb/javascript

`if` statements – one line vs. one expression

Ouverte
#445 5 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
needs eslint rule change/addition question
Langage dominant
JavaScript
Étoiles
148k
Forks
26.6k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Until I started using ESLint with your configuration I saw nothing against the rules in this:

``` js
$ cat if.js
const [one, two] = [1, 2];

if (one !== two) throw new Error(
'One does not equal two'
);
```

But ESLint does:

``` sh
$ eslint if.js

if.js
3:0 error Expected { after 'if' condition curly

✖ 1 problem (1 error, 0 warnings)
```

Is this intended? In my opinion the pattern I’ve been using is explicit – and more readable than this:

``` js
if (one !== two) {
throw new Error(
'One does not equal two'
);
}
```

The parens form a visual brace-like block much like in #438.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Le rapport identifie la règle `curly` d'ESLint et la configuration du projet, avec une comparaison avec la forme entre accolades et une référence à #438. Commencez par vérifier comment `curly` est configurée et comment le guide de style traite les instructions `if` sur une seule ligne ; le travail est terminé lorsqu'il a été décidé si la règle actuelle est intentionnelle et que cette directive a été documentée ou modifiée en conséquence.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
eslint, javascript
Domaine
documentation, tooling
Type d'issue
Fonctionnalité
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.