Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add eslint #39

Merged
merged 3 commits into from
Jul 29, 2021
Merged

chore: add eslint #39

merged 3 commits into from
Jul 29, 2021

Conversation

benjavicente
Copy link
Member

Añade eslint con las reglas por defecto y algunas reglas para forzar la consciencia en el código del proyecto (#36).

Hay que ver 2 cosas antes de aplicarlas:

  • Si creen que las reglas que elegí están bien
  • Cuáles tienen que ser warning o error, por el código de salida al ver si el código cumple las reglas (o si usar --max-warnings)

Ahora el código lo puede arreglar el linter, a excepción de 1 parte del código que se puede arreglar fácilmente.
En esta misma rama pienso modificar el código, así que una vez visto lo anterior, hago que el eslint formatee el código y habilito la PR para ser mergeada.

@benjavicente benjavicente requested review from agucova and nico-mac July 28, 2021 16:29
@benjavicente benjavicente self-assigned this Jul 28, 2021
@benjavicente benjavicente marked this pull request as ready for review July 28, 2021 22:04
Copy link
Member

@agucova agucova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creo que a futuro deberíamos poner eslint en pre-commit, pero es un buen primer paso. Me parece que los ajustes son razonables.

@benjavicente benjavicente merged commit da166ea into main Jul 29, 2021
@benjavicente benjavicente deleted the js-code-lint branch July 29, 2021 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants