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

Integrar pre-commit #11

Open
3 tasks
mdelapenya opened this issue Sep 9, 2019 · 2 comments
Open
3 tasks

Integrar pre-commit #11

mdelapenya opened this issue Sep 9, 2019 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@mdelapenya
Copy link
Member

mdelapenya commented Sep 9, 2019

Queremos automatizar todo lo posible las tareas que eliminen productividad

  • checks de formateo de código
  • Comprobar el correcto nombrado de commits
  • Lanzar los test locales

https://pre-commit.com permite hacerlo

@mdelapenya mdelapenya added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Sep 9, 2019
@mdelapenya mdelapenya mentioned this issue Sep 9, 2019
11 tasks
@mdelapenya
Copy link
Member Author

Lo de los tests locales, creo que es algo más difuso: todos los tests? sólo los unitarios?

Saquémos de esta lista los tests hasta que 1) tengamos test 2) tengamos diferentes tipos de tests a lanzar (unitarios, integración, sistemas...)

@toqueteos
Copy link

toqueteos commented Sep 10, 2019

Igual es más sencillo subir la config de VSCode y que todo el mundo use lo mismo, os evitáis un montón de trabajo (además de gestionar Python, aunque sea en un contenedor).

Si por lo que sea queréis comprobar todo igualmente, aquí van un par de sugerencias:

  • Con gofmt -d ... podeis ver un diff de los cambios por formato. Si no devuelve nada, todo en orden. Sino golpe de remo.
  • Para lints y demás, en Go lo suyo es usar golangci-lint (el sucesor de gometalinter).
  • Podéis categorizar tests con testing.Short(). Todos los short unitarios y los no short para integración y e2e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants