Library http client.po · Pull Request #1826 · python/python-docs-fr

ghost · 2022-03-23T16:04:57Z

Fin de traduction.
Première relecture faite.
C'est ma première PR, j'espère que c'est bon :-/

… je ne maîtrisais pas le vocabulaire

…10.po

jeanas · 2022-03-29T21:33:51Z

Merci pour la PR, et bienvenue. Je suis personnellement en manque de temps pour relire, j'aurai peut-être l'occasion de m'y mettre ce week-end, sauf si quelqu'un d'autre le fait dans l'intervalle.

christopheNan

Merci pour cette première contribution et le travail important de traduction effectué. Une bonne partie de mes remarques concerne la typographie ou des choix de cohérence pour l'ensemble de la traduction. Elles font généralement référence au Guide de contribution qu'il est bon de bien s'approprier.
Quelques corrections supplémentaires, reprenant les remarques apportées ici, doivent aussi être appliquées même si je ne les ai pas signalé explicitement (traduction de return par exemple).

ghost · 2022-03-30T09:39:12Z

Une bonne partie de mes remarques concerne la typographie ou des choix de cohérence pour l'ensemble de la traduction.
Elles font généralement référence au Guide de contribution qu'il est bon de bien s'approprier.

En fait, j'avais mis au programme de lecture de ce jour de document (lu en diagonale jusqu'à maintenant). Tes remarques m'aideront sûrement à me focaliser sur les points les plus courants. Merci! (Effectivement je suis tombé dans tous les pièges classiques)

Quelques corrections supplémentaires, reprenant les remarques apportées ici, doivent aussi être appliquées même si je ne les ai pas signalé explicitement (traduction de return par exemple).

Je vais essayer de les trouver tout seul (ça fera un bon exercice), par contre je ne comprends pas a priori ce que tu entends par traduction de return (mais peut être la lecture du Guide m'aidera à comprendre).

Merci en tout cas de ta patiente relecture.

Prise en compte commentaires Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

jeanas · 2022-04-05T15:55:27Z

C'est fait. J'en ai profité pour corriger quelques erreurs trouvées par pospell et padpo. Pour info, pospell doit obligatoirement passer, cela fait partie de la CI. En revanche, padpo donne encore trop de faux positifs, donc il n'est pas nécessaire de se débarasser de tous ses avertissements, même s'il reste utile de les lire. J'ai notamment mis des espaces insécables devant les deux points et au milieu des guillemets, comme préconisé par le CONTRIBUTING.rst.

@christopheNan On merge ?

ghost · 2022-04-05T15:57:19Z

Merci, j'essayerai d'être plus propre la prochaine fois...

jeanas · 2022-04-05T16:03:04Z

Ne t'inquiète pas, c'est normal pour une première contribution. On apprend au fur et à mesure.

JulienPalard · 2022-04-05T20:58:33Z

En dehors de mes 3 commentaires, c'est bon pour moi.

Merci @th71, sacrée contribution pour une première !

ghost · 2022-04-06T08:28:12Z

Un sacré monceau d'erreurs également.
Merci de votre patience ;-)

ghost

Prise en compte des remarques

Prise en compte des remarques. Co-authored-by: Julien Palard <julien@palard.fr>

christopheNan

Merci pour cette contribution. Juste quelques remarques complémentaires, toujours principalement dans un souci de cohérence avec le reste de la traduction.

Validation des propositions. Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

PyDocTeur · 2022-04-06T20:58:36Z

Hello @th71 ! Bienvenue dans le monde merveilleux de la traduction de la documentation de Python !
Je suis PyDocTeur, le robot amical qui s'occupe d'automatiser certaines tâches.
Si tu as des questions, n'hésite pas à ping @JulienPalard ou @Seluj78

Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: greetings)
PyDocTeur v1.12.0

PyDocTeur · 2022-04-06T20:58:38Z

Rajoutez-moi un automerge et ca part en prod !

Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: approved)
PyDocTeur v1.12.0

jeanas · 2022-04-06T21:38:43Z

@th71 La CI se plaint de lignes trop longues, peux-tu passer un coup de powrap library/http.client.po ?

ghost · 2022-04-07T06:46:52Z

@jean-abou-samra : juste pour comprendre (avec cette histoire de PR j'ai un peu dépasser mes connaissances en GIT). Désolé si c'est évident, mais j'ai besoin de comprendre (histoire de pas faire de c.......).

J'ai créé une divergence (fork) du dépôt officiel, pour obtenir une "copie" personnelle du dépôt qui autorise la lecture/écriture.
J'ai cloné ce dépôt distant personnel pour avoir un dépôt local où j'ai créé un branche et fait des modifs dedans
J'ai poussé cette branche dans mon dépôt distant.

[Jusque là je comprends, c'est après que ça se gâte]

Sur GitHub j'ai fait une demande de PR qui a été transmise au dépôt officiel
Diverses remarques m'ont été faites que j'ai validées : soucis: l'application de ces remarques a-t-elle modifé le code sur mon dépôt distant ou sur votre dépôt distant (j'ai du mal à voir où se passent les opérations de modifs du PR).
Tu as fait des modifs dans mon code (pour nettoyer des restes), mais comme tu n'as pas les droits d'écriture sur mon dépôt tu n'as pas pu modifier celui-ci : où donc ces modifs ont été faites? Dans un espace intermédiaire entre le dépôt officiel et mon dépôt distant? C'est assez peu clair pour moi.
Si je fais un powrap dans mon dépôt local, il sera appliqué, mais les modifications du PR non (puisque ma version locale est toujours celle du PR "originel").
Si ensuite je pousse la version "powrapée" vers mon dépôt distant puis ensuite une nouvelle PR, toutes les modifs qu'on a fait ne risquent-elles pas d'être écrasées?

Désolé de t'embêter mais comme je l'ai dis ailleurs c'est mon premier PR.

Au passage si tu as un bon doc de synthèse je suis preneur (J'ai lu il y a un certain temps ProGit v2.0. Je viens de revérifier mais il n'y a qu'un paragraphe sur les Forks qui en plus ne dit pas grand chose à part que ça existe... ce qui m'avait semblé léger à l'époque :-) )

jeanas · 2022-04-07T07:02:19Z

Il y a deux manières de modifier une PR : soit tu édites le fichier localement, soit tu appliques une suggestion dans GitHub, ce qui ajoute des commits dans la branche distante. Les deux fonctionnent, il faut juste synchroniser. Lorsque tu as fait des commits en local (dans ce dépôt nous évitons de toucher aux commits existants avec git commit --amend, nous préférons ajouter de nouveaux commits par dessus), il faut faire

git push

pour mettre à jour la branche distante sur ton fork GitHub (et donc le diff de la PR). Réciproquement, après avoir appliqué des modifications par l'interface de suggestions, fais

git pull

pour synchroniser ta branche locale. Comme tu l'as remarqué, rien n'empêche d'avoir des modifications divergentes. S'il y en a, Git te proposera une fusion, avec éventuellement des conflits. Pour simplement rewrapper les fichiers po, fais d'abord git pull, pour éviter ce genre de chose.

Non, j'ai poussé les modifications directement sur ta branche. Il y a une case « Allow edits by maintainers » que tu as cochée au moment de créer la PR.
Il ne faut pas t'inquiéter, Git est strict sur le fait de ne pas perdre de commits. Tant que tu n'utilises pas les options --force ou --force-with-lease de git push, il exigera que tes commits locaux soient fondés sur les commits distants, autrement dit qu'il n'y ait pas de commits ajoutés dans l'intervalle sur la branche distante qui soient en train d'être perdus par le push. Si cela arrive, il te suggèrera de faire git pull pour fusionner les deux historiques parallèles.

Cela ne m'étonne pas que Pro Git ne mentionne que peu les forks, puisque c'est un concept de GitHub, pas de Git à la base. Je ne connais pas trop les tutoriels Git qui existent, intéresse-toi peut-être à la documentation GitHub ?

ghost · 2022-04-07T09:15:44Z

Bon j'avais un peu pigé le truc quand même :-)

Il y a une case « Allow edits by maintainers » que tu as cochée au moment de créer la PR.

Bon à savoir!

Cela ne m'étonne pas que Pro Git ne mentionne que peu les forks, puisque c'est un concept de GitHub, pas de Git à la base.

J'apprend plein de chose aujourd'hui.

intéresse-toi peut-être à la documentation GitHub ?

oui bonne idée!

ghost · 2022-04-07T09:39:16Z

@christopheNan

th71 dismissed christopheNan’s stale review via c283378

Oups j'ai encore fait un truc qu'il ne fallait pas non?

christopheNan · 2022-04-07T12:51:27Z

@christopheNan

th71 dismissed christopheNan’s stale review via c283378

Oups j'ai encore fait un truc qu'il ne fallait pas non?

Non, ne t'inquiète pas. Si tu modifies ta PR, cela invalide automatiquement (et c'est logique) les validations que tu avais reçu.

christopheNan

Et on est tout bon !
Merci et bravo pour cette première PR.

* essai traduction * Traduction de fuzzies. Ajout de traductions manquantes * Traduction de l'introduction et de la classe HTTPConnect * Suite de la traduction. J'ai sauté un paragraphe sur la sécurité dont je ne maîtrisais pas le vocabulaire * Suite de la traduction et correction d'un barbarisme dans whatsnew/3.10.po * Suite traduction * Récupération ancienne version de whatsnew/3.10.po * Suite traduction * Fin de traduction. Attente de relecture personnelle * Début de relecture * Fin de relecture * Modif loggin * Mise à jour library/http.client.po Prise en compte commentaires Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Update library/http.client.po Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Make merge * Apply suggestions from code review Relecture finale pour corriger toutes les imperfections. Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Remise de whatsnew/3.10.po à l'état 3.10 * Remise de logging à 3.10 * powrap library/http.client.po * Faute de frappe dans http.client.po * Espaces insécables * Correction de fautes de frappe trouvées par padpo * Apply suggestions from code review Prise en compte des remarques. Co-authored-by: Julien Palard <julien@palard.fr> * Apply suggestions from code review Validation des propositions. Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> * Application de powrap Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com> Co-authored-by: Julien Palard <julien@palard.fr> Co-authored-by: Jean Abou Samra <jean@abou-samra.fr>

= added 14 commits November 15, 2021 16:47

essai traduction

da7492d

Traduction de fuzzies. Ajout de traductions manquantes

7ac4c35

Traduction de l'introduction et de la classe HTTPConnect

7e736f2

Suite de la traduction. J'ai sauté un paragraphe sur la sécurité dont…

6f5ba1e

… je ne maîtrisais pas le vocabulaire

Suite de la traduction et correction d'un barbarisme dans whatsnew/3.…

5010123

…10.po

Suite traduction

fac0d57

Suite traduction

1c06bf1

Récupération ancienne version de whatsnew/3.10.po

6d91157

Suite traduction

9aeea30

Fin de traduction. Attente de relecture personnelle

fe70f17

Début de relecture

f701175

Fin de relecture

2964588

Modif loggin

307d919

Fusions modifs suite problème merge

b9a8161

JulienPalard closed this Mar 23, 2022

JulienPalard reopened this Mar 23, 2022

christopheNan suggested changes Mar 30, 2022

View reviewed changes

= and others added 11 commits March 30, 2022 12:03

Fusion depuis upstream

32ae6bb

Mise à jour library/http.client.po

d83baa5

Prise en compte commentaires Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

9124668

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

fa5a55c

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

57c6f0f

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

23d849a

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

391f7ce

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

a0c1e32

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

bf4fbd6

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

25b1566

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Update library/http.client.po

4896612

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

jeanas added 3 commits April 5, 2022 17:38

Faute de frappe dans http.client.po

d60cfd9

Espaces insécables

f83f590

Correction de fautes de frappe trouvées par padpo

f0e06f7

JulienPalard reviewed Apr 5, 2022

View reviewed changes

Comment thread library/http.client.po Outdated

JulienPalard reviewed Apr 5, 2022

View reviewed changes

Comment thread library/http.client.po Outdated

JulienPalard reviewed Apr 5, 2022

View reviewed changes

Comment thread library/http.client.po Outdated

ghost commented Apr 6, 2022

View reviewed changes

Apply suggestions from code review

168527f

Prise en compte des remarques. Co-authored-by: Julien Palard <julien@palard.fr>

christopheNan suggested changes Apr 6, 2022

View reviewed changes

Apply suggestions from code review

3780af4

Validation des propositions. Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

christopheNan previously approved these changes Apr 6, 2022

View reviewed changes

Application de powrap

c283378

ghost dismissed christopheNan’s stale review via c283378 April 7, 2022 09:16

christopheNan approved these changes Apr 7, 2022

View reviewed changes

christopheNan merged commit 5460f47 into python:3.10 Apr 7, 2022

Uh oh!

Uh oh!

Conversation

ghost commented Mar 23, 2022 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jeanas commented Mar 29, 2022

Uh oh!

christopheNan left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ghost commented Mar 30, 2022 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jeanas commented Apr 5, 2022

Uh oh!

ghost commented Apr 5, 2022

Uh oh!

jeanas commented Apr 5, 2022

Uh oh!

Uh oh!

Uh oh!

Uh oh!

JulienPalard commented Apr 5, 2022

Uh oh!

ghost commented Apr 6, 2022

Uh oh!

ghost left a comment

Choose a reason for hiding this comment

Uh oh!

christopheNan left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

PyDocTeur commented Apr 6, 2022

Uh oh!

PyDocTeur commented Apr 6, 2022

Uh oh!

jeanas commented Apr 6, 2022

Uh oh!

ghost commented Apr 7, 2022 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jeanas commented Apr 7, 2022

Uh oh!

ghost commented Apr 7, 2022 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ghost commented Apr 7, 2022

Uh oh!

christopheNan commented Apr 7, 2022

Uh oh!

christopheNan left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

ghost commented Mar 23, 2022 •

edited by ghost

Loading

ghost commented Mar 30, 2022 •

edited by ghost

Loading

ghost commented Apr 7, 2022 •

edited by ghost

Loading

ghost commented Apr 7, 2022 •

edited by ghost

Loading