Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Translation of tarfile.po from english to french #1274

Merged
merged 32 commits into from Aug 8, 2020

Conversation

Sim4n6
Copy link
Contributor

@Sim4n6 Sim4n6 commented May 9, 2020

Closes #1031.
Etant donné que j'ai quelques "free automatic translations" on poeditor.com, j'ai procédé à la tranduction de tarfile.po 👍 Je suis disponible pour vos remarques afin d'améliorer la traduction.

@Sim4n6 Sim4n6 changed the title Translation for tarfile.po from english to french Translation of tarfile.po from english to french May 9, 2020
@awecx
Copy link
Contributor

awecx commented May 10, 2020

Merci pour cette PR @Sim4n6.

Néanmoins elle ne respecte pas les règles définies dans le guide. Notamment sur la syntaxe (ce qui a cassé le build) :

  • l'indentation n'est pas respectée, @deronnax l'a corrigée pour toi ;
  • des « ` » ont été changés en « ' » ;
  • l'ajout d'espaces inutiles a cassé certaines balises.

Tu as utilisé un traducteur automatique. Nous le déconseillons aussi ; les traductions générées sont très souvent à retravailler. Certaines phrases de ta PR n'ont ainsi pas de sens en français. Nous avons aussi défini certains termes dont nous imposons une traduction (p. e. « renvoie » pour « return » et non pas « retourne »). Il faut respecter ces règles. Le but de notre travail n'est pas de brancher un traducteur sur les fichiers de la documentation (auquel cas cela fait longtemps que toute la documentation aurait été traduite), mais de produire une documentation correcte et agréable à lire.

Je te laisse modifier ta PR avant de la relire.

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 10, 2020

Merci pour votre réponse. Je vais lire le guide et voir ce qui peut etre rectifier.

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 12, 2020

I really don't get this one NBSP error. There should be a non-breakable space before ":"
Any help please ?

@deronnax
Copy link
Sponsor Collaborator

ça veut dire que padpo veut une espace insécable avant les : , mais c'est pas une erreur importante. Ce que remonte padpo ne sont que des warning, ce n'est pas obligatoire de les corriger pour que les tests automatiques passent au vert (la petite croix rouge)

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 12, 2020

Désolé les gars, mais encore une fois je bloque :) help wanted

@deronnax
Copy link
Sponsor Collaborator

tu bloques sur quoi ?

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 13, 2020

Checks keep failing... I made the necessary changes but still.

@deronnax deronnax self-assigned this May 14, 2020
@awecx
Copy link
Contributor

awecx commented May 16, 2020

Checks keep failing... I made the necessary changes but still.

Regarde le journal de la compilation : https://travis-ci.org/github/python/python-docs-fr/builds/686308381.
La compilation échoue car tu as énormément de termes non-traduits ou mal traduits. Par exemple,
l. 46 « ecriture » => « écriture »
l. 700 « headers » => « en-têtes »
l. 593 => probablement un « ' » manquant ou un « ` » qui se serait transfromé en « ' ».
Il faut mettre les termes qu'on ne traduit pas (par exemple gzip entre astérisques).

Enfin, attention au sens. J'ai parcouru ta PR en diagonale et je vois beaucoup de temps mal traduits.
Par exemple, "Create a tarfile with compression…" se traduit en « Crée un fichier tar compressé avec… » et non « Créez » ni « Créer ».

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 16, 2020

Finally, all checks have successfully succeeded :)
Tomorrow, I will re-read everything and make a last modif 👍

@awecx
Copy link
Contributor

awecx commented May 17, 2020

Entendu.
N'oublie pas de lire le glossaire de la traduction afin de respecter les conventions utilisées dans la traduction (p. ex. tu as traduis « support » par « supporte » là où il aurait fallu mettre « prend en charge ».

@deronnax
Copy link
Sponsor Collaborator

Hello @Sim4n6 . Tout d'abord, merci et bravo pour ta persévérance. Tu as fait quelque chose qui va poser une problème : tu as mis entre astérisques même les chaines d'origine en anglais (celles dont le préfix est msgid), comme la ligne 47 ou 62. Ces lignes sont les lignes d'origine de la doc, et c'est elles qui seront recherchées par sphinx pour les remplacer par la traduction en dessous. Il ne faut donc pas les changer sinon elles ne seront plus trouvées.
Je te laisse corriger ça, et merci encore pour ton dévouement :)

@deronnax deronnax removed their assignment May 18, 2020
@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 18, 2020 via email

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 18, 2020

Salut, vous trouverez dans ce PR la meilleure version possible de la traduction de tarfile.po.
J'avais commis quelques erreurs, au fait j'aurais du set-up l'environnement depuis le début mais bon. J'apprends 👍

Copy link
Contributor

@awecx awecx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai commencé à relire ta PR ; je vois des erreurs.

En particulier, il faut prendre garde à ne pas changer le texte original des msgid, faire attention aux temps des verbes et bien comprendre le sens de ce que tu traduis (il faut que tu comprennes le but du module tarfile).

Le guide de la traduction a été récemment mis à jour, je t'enjoins à le relier, tu y trouveras des explications sur comment compiler la documentation sur ta machine et l'afficher. Tu pourras alors relire tes traductions et vérifier le sens de tes traductions.

library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
library/tarfile.po Outdated Show resolved Hide resolved
@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 23, 2020

Bon, je me demande si après avoir appliqué votre review de la traduction, si on procède au merge du PR ou je continue à le peaufiner ?

@awecx
Copy link
Contributor

awecx commented May 24, 2020

Bon, je me demande si après avoir appliqué votre review de la traduction, si on procède au merge du PR ou je continue à le peaufiner ?

Relis-toi d'abord. En particulier je vois encore beaucoup de msgid modifiés, il faut les rétablir à leur version d'origine.

@deronnax
Copy link
Sponsor Collaborator

deronnax commented May 24, 2020

Hello @Sim4n6. Ça commence à être pas mal mais y'a encore des trucs qui font qu'on ne peut pas la prendre en l'état. En effet comme le dit Antoine, il reste encore quelques msgid modifiés (les chaines d'origine).
Et aussi, as-tu essayé de compiler la traduction sur ton PC, pour afficher le résultat ?

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented May 24, 2020

Okey no problem, je vais my mettre des que possible...

@deronnax
Copy link
Sponsor Collaborator

deronnax commented Jun 2, 2020

Allez je m'occupe de restaurer les msgstr, ça je peux le faire. @Sim4n6 , si tu as du travail en cours, fait un commit "WIP" et pousse-le, sinon quand tu vas récupérer ma modif ça va faire des conflits.

@Sim4n6
Copy link
Contributor Author

Sim4n6 commented Jun 2, 2020

Vas-y je n'ai pas fait de modifs depuis.

@Sim4n6 Sim4n6 requested a review from awecx June 8, 2020 09:38
@JulienPalard JulienPalard force-pushed the library-tarfile-po branch 3 times, most recently from b5c0b2b to 63b6ed0 Compare June 25, 2020 11:42
Sim4n6 and others added 21 commits August 8, 2020 11:48
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@JulienPalard
Copy link
Member

J'ai rebasé ça sur la 3.9 à la main car github n'y arrivait pas. LGTM.

@deronnax
Copy link
Sponsor Collaborator

deronnax commented Aug 8, 2020

Fursion.
Et bien c'est l'une des plus grosse PR jamais faite sur python-docs-fr qui est fusionnée. Merci beaucoup à @Sim4n6 pour avoir patiemment intégré les milliers de suggestions successives sans jamais avoir baissé les bras, et merci beaucoup à @awecx @christopheNan et @JulienPalard pour ces revues de titans.
Bravo à tous.

@deronnax deronnax merged commit 274e123 into python:3.9 Aug 8, 2020
@JulienPalard
Copy link
Member

C'est en effet une des plus grosses PR, à égalité (a la louche) avec #1163 et #1020.

Bravo.

(Je reste persuadé qu'il est plus agréable de travailler avec des PR plus petites :p)

deronnax added a commit to deronnax/python-docs-fr that referenced this pull request Aug 9, 2020
* Translation for tarfile.po from english to french

* Fix to make serve working correctly

* Text improvement

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>

* update after re-reading using Poeditor

* Small fix wrap and spell

* Small fix for a runaway space

* espaces insécable + minuscules pour certaines phrases

* Apply suggestions from code review

Co-authored-by: Julien Palard <julien@palard.fr>

* Apply suggestions from code review

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

* Fix of wrapping

* ajoute un marqueur de fuzzy

* Useless spaces.

Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
deronnax added a commit that referenced this pull request Aug 9, 2020
* Translation for tarfile.po from english to french

* Fix to make serve working correctly

* Text improvement

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Julien Palard <julien@palard.fr>

* Update library/tarfile.po

Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>

* update after re-reading using Poeditor

* Small fix wrap and spell

* Small fix for a runaway space

* espaces insécable + minuscules pour certaines phrases

* Apply suggestions from code review

Co-authored-by: Julien Palard <julien@palard.fr>

* Apply suggestions from code review

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

* Fix of wrapping

* ajoute un marqueur de fuzzy

* Useless spaces.

Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

Co-authored-by: Sim4n6 <sim4n6@gmail.com>
Co-authored-by: Julien Palard <julien@palard.fr>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Je travaille sur library/tarfile.po
5 participants