Make wrap to actually wrap, not just check.#1765
Conversation
JulienPalard
force-pushed
the
mdk-make-wrap
branch
from
7766d53 to
8cff526
Compare
There was a problem hiding this comment.
J'ai dit une bêtise. Il ne faut pas le faire avec -m car nous utilisons tous couramment powrap sur le résultat d'un commit fait sur GitHub, et powrap ne considère pas cela comme un changement dans le dépôt (contrairement à make spell qui regarde la différence avec BRANCH). Donc :
| powrap -m | |
| powrap --quiet *.po **/*.po |
There was a problem hiding this comment.
Je pense que la partie make checks / make wrap est plutôt dédiée aux nouveaux. Les anciens connaissent par coeur toutes les options de powrap et n'ont pas besoin du Makefile.
En passant je préfère powrap *.po */*.po à powrap **/*.po, le second risque de trouver des fichiers po dans le venv.
Si vraiment ça te manque, tu pourrais te rajouter make wrapall ?
|
J'ai raccourci le CONTRIBUTING, dans un souci de le garder lisible et digeste. |
4 participants
Reprise de #1764 cc @Fipaddict @jean-abou-samra