Produits Entr'ouvert » Authentic 2

Ça donne l'idée bizarre qu'il faut contribuer des .po actualisés mais ne contenant pas les nouvelles traductions.

L'autre possibilité est de ne pas actualiser les .po avec les nouvelles chaines du tout. Mais c'est prendre le risque qu'une erreur du genre `_("foo {bar}".format(bar=1))` au lieu `_("foo {bar}").format(bar=1)` de passe inaperçue. Alors qu'elle se voit assez bien dans le diff du .po au moment du review, ce qui me motive a mettre à jour les .po histoire d'éviter ce genre d'accident.

Ceci dit c'est juste mon avis et si la consigne est plutôt de ne jamais le faire, ça me va aussi: moins de travail pour le contributeur je suis pas contre :-)

Je voulais dire

au moment de la revue de code

et non pas

au moment de la review

Pas de modif au .po dans les patchs; ça rend tout travail de merge/cherry-picking/rebase/etc. pénible.

as seulement

doit se lire

a seulement

Je ne pense pas que ce soit la place du README ...

Je pense qu'il y a une confusion: il s'agit de la section développement de la documentation d'Authentic et non du README (ce serait un peu superflu dans un README, en effet).

Je maintiens tout de même ma remarque.

L'objet de cette section est double:

• Documenter la façon dont il est possible de mettre à jour les traductions (ce n'est pas trivial)
• Préciser que c'est fait ailleurs

Si cette section se contentait de mentionner que les traductions sont mises à jour ailleurs, sans préciser comment techniquement, cela ferait défaut à une personne qui voudrait faire elle même les traductions, quelque soient ses motifs. C'est une information utile, qui se trouve dans le bon contexte et qu'on ne trouve pas ailleurs.

Documenter la façon dont il est possible de mettre à jour les traductions (ce n'est pas trivial)

Et les instructions dans le patch ne sont pas correctes.

Il ne fonctionne peut-être plus, ou est lourdingue, mais il y a update-locales.sh ajouté là :

commit e0071fb470536ce0f38f576d7cd75191e99ed924
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Jun 22 16:03:35 2019 +0200

    add script to update locales

Je verrais bien une réorganisation de la section pour dire d'abord qu'il ne faut pas commiter de trad, et ensuite pointer la commande pour au cas où, tout en enlevant le bout qui raconte que c'est « useful to check for mistakes ».