fix translation howto/unicode.po#1094
Conversation
awecx
left a comment
awecx
left a comment
There was a problem hiding this comment.
Merci pour cette PR. Je t'ai mis quelques remarques que tu peux accepter ou non ; ta traduction est cohérente. Je fais confiance à padpo pour les espaces insécables.
Par contre, il te faut réindenter le fichier (avec powrap comme expliqué dans le guide) pour accepter ta PR.
|
Ping :-) |
Remplacement de "HOWTO" par "guide" Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Annulation de la mise en italique interne à un lien Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
|
Désolé pour le temps d'attente. J'ai commité les premieres propositions. Je vais voir en quoi l'usage de powrap consiste pour pouvoir réindenter. Bonne soirée |
There was a problem hiding this comment.
Désolé pour le temps d'attente. J'ai commité les premieres propositions.
Pas de soucis. Merci.
Je vais voir en quoi l'usage de powrap consiste pour pouvoir réindenter.
Tout d'abord, fais un git pull pour récupérer les modifications effectuées sur Github, dans ta copie locale.
powrap est dans le venv/ de ta copie (tu peux aussi l'installer depuis Pypi et le lancer sans environnment virtuel).
source venv/bin/activate
powrap howto/unicode.po
deactivate
et tu repushs les modifications éventuelles.
Une question de débutant : quand on fait un commit en acceptant des modifications proposées, cela met-il bien a jour l'ensemble de la pull request?
Oui. D'ailleurs n'hésite pas à faire un seul commit de plusieurs modifications avec le add suggestion to batch. Ça fait moins de commits (et je reçois moins de courriels de notifications 😃).
changement mise en forme Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
|
Oups desole pour les notifications! je crois avoir tout réglé. Voyez vous d'autres choses? Merci! |
En tout cas, il y a marqué « 1 successful check », ce qui signifie qu'entre autres powrap est content pour la réindentation. |
awecx
left a comment
awecx
left a comment
There was a problem hiding this comment.
Merci pour les corrections apportées.
Je t'en ai fait 3 autres après une seconde lecture. N'oublie pas de réindenter le fichier après les avoir réglées.
Merci.
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
|
J'ai accepté les modifications. Il me reste à réindenter. Merci. |
|
@massicotjgab il te manque un |
|
Hello @massicotjgab . Coche la case "allow edits from mainteners" en bas du panneau à droite stp, je m'occupe du reste. |
|
@deronnax c'est fait, désolé pour les délais, merci! |
|
merci beaucoup. Je te fais les corrections et fusionne ça sous peu |
|
@awecx faut que tu ré-approuves. On a basculé sans faire gaffe dans un nouveau workflow Github où, entre-autre, on peut pas merger si le mec qui a fait des remarques a pas re-approuvé ensuite, c'est ultra-rigide, ultra-lourd, faut repasser à l'ancien d'urgence. |
|
Je suis d'accord avec @deronnax , l'ancien workflow me convenait mieux. |
|
Quand vous me voyez fermer une PR pour la rouvrir juste après, c'est pour relancer la CI. |
|
Je merge mais il y a quand même beaucoup de choses trucs qui me dérangent sur cette PR :
|
On avait pas dit, au contraire, on laisse le choix au traducteur (à condition de ne pas faire des PRs uniquement pour ça) ?
Ça reste un terme anglais, même s'il est dans la liste blanche.
Pas grave ça. |
En fait, cette PR date du temps où ça n'avait pas encore été statué ! Et comme visiblement @awecx et @deronnax vous n'avez pas le même statut, la décision (toujours pas mise dans padpo, j'ai un planning de fou en ce moment) est ici : https://github.com/python/python-docs-fr/pull/1113/files
|
|
Ah, très bien vu. Merci @vpoulailleau |
6 participants
Fix #1086
First pull request.