Automatic `tox -e format` i `tox`

[nabiu256]

Havent implementat el tox i fent proves he notat que és molt fàcil que se'm passi executar les comandes tox -e format i tox abans de fer cada push, amb el que llavors les github actions fallen i haig de fer un commit i push extra abans de fer merge de la branca.

He recordat que l'altre dia vaig descobrir que git té hooks, i he vist que implementar .git/hooks/pre-push amb un

#!/bin/sh
eval "tox -e format"
eval "tox"

permet que git mateix executi aquestes comandes automàticament cada cop que intentis fer push, i si qualsevol de les comandes torna un codi d'error, llavors l'aborta. Crec que seria prou interessant afegir això d'alguna manera a les boilerplates per tal de fer-nos la feina més fàcil, i et volia demanar què en penses.

[perepicornell]

No coneixia això, és xulo, si ho vols provar i funciona bé fantàstic. Els dubtes que em genera son:

Pel que fa al -e format:

• Realment no arregla tot el format, hi ha coses que t'avisarà quan facis tox i que s'han de fer a ma, p.ex. imports o variables que no es facin servir.

Pel que fa al tox:

• Quan github executa l'action tarda MOLT, és exagerat xD si al fer push ha de fer el mateix procés que fa ara, serà un problema. Diferent serà si aquests hooks no els fa en remot sinó que els executa en local, en tal cas entenc que sí que tirarà bé.

Encantat de fer proves quan em diguis!

[nabiu256]

Pel tema de tox -e format, la gràcia és que el hook fa que s'aborti el push en cas que qualsevol part de l'script retorni un codi d'error, cosa que vol dir que si el tox et falla, t'aborta el push i pots veure per terminal quin ha estat el problema, solucionar-ho, i tornar-li a donar push.

Pel que fa al tox, just lo bo és que és una cosa de git, no de github, i per tant efectivament, l'script s'executa a la teva màquina, amb el que és un procés bastant ràpid, molt més que quan ho fa github, que com tu dius tarda moltíssim.

Si vols ara si ens reunim una estona t'ho ensenyo a la meva màquina a veure què et sembla.