airbnb / airbnb/javascript

`if` statements – one line vs. one expression

Abierto
#445 5 comentarios 0 reacciones 0 asignados Ver en GitHub
needs eslint rule change/addition question
Lenguaje dominante
JavaScript
Estrellas
148k
Forks
26.6k
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

El informe identifica la regla `curly` de ESLint y la configuración del proyecto, con una comparación con la forma con llaves y una referencia a #438. Empieza comprobando cómo está configurada `curly` y cómo aborda la guía de estilo las sentencias `if` de una sola línea; el trabajo estará terminado cuando se haya decidido si la regla actual es intencionada y se haya documentado o cambiado esa guía en consecuencia.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
eslint, javascript
Área
documentation, tooling
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.