Make wrap to actually wrap, not just check. #1765
Conversation
7766d53 to
8cff526
Compare
| @echo "Verify wrapping" | ||
| powrap --check --quiet *.po **/*.po | ||
| @echo "Re wrapping modified files" | ||
| powrap -m |
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. |
|
Hello @JulienPalard ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter. DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: incorrect_title) |
Reprise de #1764 cc @Fipaddict @jean-abou-samra