Hello @lambelin ! 🏓

@Seluj78 Qu'est-ce qu'on attend de moi là-dessus ? Que je mette un +1 sur la PR en l'état ou que je fasse une nouvelle PR par dessus ?

@entwanne la première personne qui commente une PR en deviens responsable :) Du coup c'est a toi de commenter, ping, merge et reprendre si besoin ;)

Ok. Du nouveau @lambelin ?
Sinon j'ajoute la suggestion que j'ai faite et je merge en l'état.

Je vois que travis échoue le build parce que pofile.py indique qu'il devrait y avoir une espace insécable avant un :ref: (à cause du « : »). On est d'accord que c'est une erreur du programme ?

Je vois que travis échoue le build parce que pofile.py indique qu'il devrait y avoir une espace insécable avant un :ref: (à cause du « : »). On est d'accord que c'est une erreur du programme ?

padpo est un outil en cours de développement (disclaimer : par moi 😉), et donc les messages renvoyés par padpo ne sont pas encore considérés comme une source d'échec de build, ils sont juste informatifs.

En fait, le build échoue avant :

Would rewrap:

- library/warnings.po

Ce qui nécessite un coup de powrap.

Le message que tu mentionnes est en fait lié à une erreur précédente qui perd padpo dans son analyse :

:exc:`Warning'

Devrait être

:exc:`Warning`

(à la ligne 779)

Et du coup padpo pense que le :exc: se termine au début du :ref: mentionné dans le message d'erreur…

Donc en résumé, il faut :

• corriger l'apostrophe
• ré-indenter le fichier avec powrap

Ok pour moi, j'attends une seconde approbation ou je peux merger ?

Non la règle c'est au moins une approbation (sauf sur sa propre PR, naturellement).
Tu peux y aller.

Merci pour cette PR.

Non la règle c'est au moins une approbation (sauf sur sa propre PR, naturellement).
Tu peux y aller.

Ok, mais comme j'étais l'auteur des fix pour la CI je n'étais pas sûr. Merci.

Et merci @christopheNan pour le merge !