Less.js: [Tarea de mantenimiento]: revise la configuración de ESLint y solucione los problemas de formación de pelusa
Hace un tiempo cambié el proyecto de JSHint a ESLint (este último está mejor mantenido y ofrece más funciones). Sin embargo, configuré muchas de las configuraciones como advertencias porque no tuve tiempo de abordar problemas potenciales. Tampoco tuve tiempo de modificar el linting para configuraciones que hacen que el código sea más fácil de mantener.
Entonces, la tarea sería esencialmente mirar las advertencias de ESLint y ver qué se puede abordar de manera segura. (Por ejemplo, muchas de las advertencias de igualdad == pueden no ser "reparables" en === sin probar cada valor que pasa por esa verificación). Sería útil establecer también advertencias para cosas para abordar en el futuro / en curso, como hacer cumplir la documentación JSDoc.
matthew-dean
Todos 6 comentarios
¡Hola, me gustaría agarrar esto!
umuur
en 18 ene. 2021
@umuur ¡ Adelante!
matthew-dean
en 19 ene. 2021
@ matthew-dean ¿Queremos mantener la configuración de eslint actual?
No estoy seguro de que sea necesario tener TypeScript como analizador y complemento.
umuur
en 19 ene. 2021
@umuur
¿Queremos mantener la configuración de eslint actual?
Depende. Si desea realizar modificaciones que no cambien el formato del código, está bien. Pero en cuanto a las advertencias, en mi opinión, todas son advertencias válidas y queremos abordarlas o dejar un comentario de código sobre el eslint deshabilitado y (lo más importante ) por qué no es válido allí.
¿Queremos mantener la configuración de eslint actual?
Aunque el código base todavía no usa TypeScript, en mi experiencia, hace un mejor trabajo de transpilación que Babel. En términos de ESLint ... 🤔 sí, tiene razón, técnicamente no es necesario hasta que haya código en TypeScript. Probablemente hice eso antes de convertir el código base a TS, pero ahora no sé si o cuándo sucederá.
matthew-dean
en 20 ene. 2021
@umuur
Por cierto, en cualquier lugar donde pueda agregar comentarios JSDoc con los tipos adecuados en los parámetros, ¡agréguelos!
matthew-dean
en 20 ene. 2021
@ matthew-dean ¡gracias por los comentarios detallados! Le informará sobre las actualizaciones.
umuur
en 20 ene. 2021
Temas relacionados
falkenhawk
·
5Comentarios
yggi49
·
7Comentarios
xblakestone
·
3Comentarios
pknepper
·
3Comentarios
renoth
·
6Comentarios