Traduction de library/heapq.po #879
Conversation
|
Quelques petits soucis dans la CI : les smileys ne passent pas. Plusieurs choix :
|
|
Je serais presque d'avis de retirer les smileys mais c'est un choix éditorial un peu fort. Va pour les remplacer par ☺. |
Modifs pour passer la CI appliquées, mais je n'ai pas commencé à relire.
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
|
@nshaud faut que tu refasse le wrapping du fichier, parce que les suggestions via Github cassent le wrapping (et du coup la CI le détecte et met le build en erreur) |
Bien vu, c'est fait. |
|
question Github : pourquoi y'a toujours "change requested" qui apparaît à coté de la PR même quand l'auteur a appliqué toutes les recommandations, marqués tous les commentaires comme résolus ? C'est toujours comme ça ? Y'a que moi que ça choque ? |
Il me semble que c'est toujours comme ça. Si j'ai bien compris, l'auteur peut clore les conservations (et les marquer comme résolues) mais il faut que le relecteur de la PR valide que les changements ont bien appliqués. |
|
Ah ! Merci beaucoup !
Le lun. 23 sept. 2019 à 14:34, Nicolas <notifications@github.com> a écrit :
… question Github : pourquoi y'a toujours "change requested" qui apparaît à
coté de la PR même quand l'auteur a appliqué toutes les recommandations,
marqués tous les commentaires comme résolus ? C'est toujours comme ça ? Y'a
que moi que ça choque ?
Il me semble que c'est toujours comme ça. Si j'ai bien compris, l'auteur
peut clore les conservations (et les marquer comme résolues) mais il faut
que le relecteur de la PR valide que les changements ont bien appliqués.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#879?email_source=notifications&email_token=AADLH3Z23PSHXQWXOVTEFNTQLCZTPA5CNFSM4IXBM2EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7KWK7I#issuecomment-534078845>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADLH34YPQVY4UANUL5YTZDQLCZTPANCNFSM4IXBM2EA>
.
|
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
|
Pour nouveau passage dans Travis. |
|
Let's go. |
|
Un wrapping est nécessaire avant la fusion. |
|
Fait, Travis devrait être content. |
Module de la bibliothèque standard qui implémente les tas et les files de priorité.
Je ne suis vraiment pas très satisfait de la documentation (notamment à partir des Basic Examples) mais c'est un problème upstream... Il y aurait pas mal à faire pour la rendre un peu plus pédagogique, mais bon, en attendant voici une proposition de traduction.
Au risque d'aller contre les bonnes pratiques, j'ai choisi de traduire le pseudo-code pour le premier paragraphe parce que ça me semble plus clair.