search

FdelMazo / FIUBA-Map

Mapa de materias y sus correlativas de la Facultad de Ingeniería
https://fede.dm/FIUBA-Map/
MIT License
126 stars 38 forks source link

Implement code formatter #201

Closed lopezac closed 5 months ago

lopezac commented 5 months ago

Hola, me parece un formateador de codigo serviria mucho para que el codigo quede mas ordenado y legible.

Prettier me parece una muy buena opcion.

No sabria si harian falta alguna set de rules o con las reglas por defecto esta bien.

Por ejemplo el archivo de body/Controls.js termina aumentadole 22 lineas, y para un archivo mas grande como Graph.js termina agregando 112 lineas, paso unos pastebin de body/Controls.js sin prettier, y con prettier.

FdelMazo commented 5 months ago

Buenas! Me parece genial la idea. Prettier parece ser lo mas standard, y creo que por ahora es mejor salir sin set de reglas.

Gracias por el diff de con y sin prettier!

lopezac commented 5 months ago

Te parece bien que haga un pull request instalando prettier al proyecto y aplicandole prettier a todos los archivos del proyecto?.

FdelMazo commented 5 months ago

Si! Impecable. Gracias por la idea!

lopezac commented 5 months ago

Te pareceria bien instalar y aplicar prettier tambien en fiuba-repos y fiuba-plan?. @FdelMazo

FdelMazo commented 5 months ago

recontra! todo ayuda