Openlibrary: ¡Vamos a lustrar nuestro JavaScript con eslint!
Stylelint nos protege de varios problemas de CSS. Hagamos lo mismo con nuestro JS.
eslint es el más maduro y proporciona una lista de reglas que podemos utilizar si lo consideramos necesario.
npm install eslint --save-dev
En package.json, agregue un nuevo comando
"lint:js": "eslint openlibrary/plugins/openlibrary/js/"
Agregue un .eslintrc.json
{
"root": true,
"env": {},
"globals": {
},
"rules": {
"no-extra-semi": "warn"
}
}
Configuraremos los errores para advertir por el momento y tan pronto como la configuración se fusione, deberíamos coordinarnos con alguien para ejecutar las correcciones y fusionar para evitar cualquier conflicto de fusión.
jdlrobson
Todos 5 comentarios
Hola. ¿Me pueden asignar para resolver este problema?
0x15F9
en 18 oct. 2018
Pregunta: ¿por qué no utilizar uno de los ajustes preestablecidos por eslint ? Como https://standardjs.com o https://github.com/airbnb/javascript
tamtamchik
en 20 oct. 2018
A largo plazo, me gustaría, pero el javascript está en tan mal estado, aplicar un ajuste preestablecido de este tipo sería muy confuso (revisión de código, conflictos de rebase) y trabajo manual (la mayoría de los ajustes preestablecidos no se pueden arreglar automáticamente). Creo que agregar reglas manualmente nos ayuda a atacarlo poco a poco de una manera cohesiva y educativa (aprendemos sobre los problemas en nuestro código poco a poco).
Pensé que podríamos usar la pelusa para solucionar los problemas más comunes y urgentes (las pelusas que pueden causar errores en el código) y luego, en nuestro tiempo libre, aplicar un ajuste preestablecido (de nuestra elección) más adelante.
¿Esto tiene sentido?
jdlrobson
en 20 oct. 2018
Claro que sí. La forma del código puede ser muy mala.
Cuando este PR se fusione, puedo solucionar algunos problemas con el borrado, solo nómbrelos.
Comenzará con no-extra-semi por ejemplo;)
tamtamchik
en 20 oct. 2018
@tamtamchik puedo crear un problema más tarde, pero cualquier cosa que esté bajo "posibles errores" y que se pueda arreglar automáticamente es un juego limpio.
Deberíamos coordinarnos con @mekarpeles para ejecutar y confirmar las correcciones automáticas y hacer un error de eslint en lugar de advertir.
¡Quiero que nos deshagamos de la fruta madura!
jdlrobson
en 21 oct. 2018
Temas relacionados
BrittanyBunk
·
5Comentarios
cdrini
·
6Comentarios
cdrini
·
4Comentarios
cclauss
·
3Comentarios
cdrini
·
5Comentarios