`if` statements – one line vs. one expression
- 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