search

lugfi / organizador-fiuba

El organizador te ayuda a organizar las materias con un calendario semanal, mostrando superposiciones y permitiendo probar distintas combinaciones de cursos rapidamente.
https://lugfi.github.io/organizador-fiuba/
GNU General Public License v3.0
15 stars 16 forks source link

Agregar Eslint para mejorar el código js #24

Closed edittler closed 7 years ago

edittler commented 7 years ago

Agregué el package.json con eslint para que cualquiera pueda correrlo fácilmente (con npm run lint) y tratar subir código sin errores. Actualmente figuran muchos "errores" y warnings pero habría que eliminarlos de a poco. Inclusive se podría configurar travis para que lo corra y reporte el resultado de eslint en los pull requests.

colltoaction commented 7 years ago

Creo que si lo ponemos y tiene errores, no va a ser de gran ayuda. Nadie va a correrlo e intentar diferenciar sus errores de los preexistentes.

Por mí banquemos hasta que alguien quiera corregir los errores como parte de este PR. Pero si realmente quieren mergear no tengo problema, no me parece contraproducente.

AlexDaciuk commented 7 years ago

En teoria este cuatri arrancan con el proyecto de la reescritura del organizador, no veo que valga la pena parchear este ... bah @tinchou es quien va a estar a cargo del proyecto ese, creo que decide el sobre ese tema

colltoaction commented 7 years ago

Se podría mergear y que quede para el futuro. El que quiera lo usará.

On Wed, Aug 2, 2017, 12:33 PM Alexis Daciuk notifications@github.com wrote:

En teoria este cuatri arrancan con el proyecto de la reescritura del organizador, no veo que valga la pena parchear este ... bah @tinchou https://github.com/tinchou es quien va a estar a cargo del proyecto ese, creo que decide el sobre ese tema

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/lugfi/organizador-fiuba/pull/24#issuecomment-319774783, or mute the thread https://github.com/notifications/unsubscribe-auth/ABeg9Ng8E4cQBTbKvkDh4koEfzZWQgNLks5sUM7ygaJpZM4Oq9vb .

edittler commented 7 years ago

Me puedo comprometer a bajar los warnings del proyecto, sin tests y todo trataré de hacerlo a conciencia jajaja

Podríamos hacer lo que dice tincho, se aprueba este PR y queda a libre disponibilidad de quien quiera usarlo. Lo bueno es que con sólo 2 comandos ya puede correrlo npm install y npm run lint (quizás haya que agregar esto en el README o inicializar el archivo CONTRIBUTING.md para meter todos estos detalles.

Además, se pueden seguir agregando tasks al package.json como la de reducir los jsons o tareas similares.

colltoaction commented 7 years ago

Gracias!