Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Translation of distutils/setupscript.po #1163

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

Merged
merged 30 commits into from
May 5, 2020

Conversation

PhilippeGalvan
Copy link
Contributor

closes #1144

Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

Je t'ai fais une ou deux proposition de modification pour que tu vois le type de problème, je te laisse faire les autres ? N'hésite pas si tu as besoin d'aide.

@JulienPalard JulienPalard self-assigned this Feb 23, 2020
@PhilippeGalvan
Copy link
Contributor Author

J'ai pris en compte l'ensemble des remarques au dessus, j'en ai profité pour me faire taper sur les doigts par padpo et réviser mon français à la volée. Il me reste la question des " non-breakable space " dans poedit, je ne suis pas trop certain de comment les ajouter dans cet éditeur, si quelqu'un peut m'aider sur ce point je suis preneur :) !

@JulienPalard
Copy link
Member

Il me reste la question des " non-breakable space " dans poedit, je ne suis pas trop certain de comment les ajouter dans cet éditeur, si quelqu'un peut m'aider sur ce point je suis preneur :) !

Est-ce que ce chapitre : https://github.com/python/python-docs-fr/blob/3.8/CONTRIBUTING.rst#caract%C3%A8res-sp%C3%A9ciaux suffit ?

@PhilippeGalvan
Copy link
Contributor Author

PhilippeGalvan commented Feb 26, 2020

Effectivement j'avais vu cette partie dans la doc mais j'avoue être assez frileux sur l'utilisation de lignes de commandes sans trop comprendre ce qu'elles font. Au final, j'ai quand même réussi à trouver l'interface graphique pour le faire grâce à ce thread : https://unix.stackexchange.com/questions/474480/how-to-disable-compose-key-on-fedora-28 en récupérant "GNOME TWEAKS" :) ! Ce coup ci padpo a l'air d'être plutôt d'accord avec ma proposition de traduction, je pense qu'il y a sans doute des choses à améliorer mais il y a eu un du ménage par rapport à la précédente version !

@PhilippeGalvan
Copy link
Contributor Author

PhilippeGalvan commented Feb 26, 2020

J'ai essayé de corriger au maximum les erreurs diverses, de grammaire, conjugaison, syntaxe. Je pense que les dernières erreurs sont du vocabulaire :

distutils/setupscript.po:436:X11R6
distutils/setupscript.po:968:Distutils
distutils/setupscript.po:1218:distutils
distutils/setupscript.po:1298:a1
distutils/setupscript.po:1298:a2
distutils/setupscript.po:1298:b1
distutils/setupscript.po:1298:b2
distutils/setupscript.po:1298:bN
distutils/setupscript.po:1298:beta
distutils/setupscript.po:1298:pr1
distutils/setupscript.po:1298:pr2
distutils/setupscript.po:1298:prN
distutils/setupscript.po:1366:Distutils
distutils/setupscript.po:1385:distutils

PhilippeGalvan and others added 8 commits March 11, 2020 19:23
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
@PhilippeGalvan PhilippeGalvan force-pushed the distutils_setupscript.po branch from a9c98a8 to d3b8a6a Compare March 11, 2020 22:42
@PhilippeGalvan
Copy link
Contributor Author

Merci encore pour la relecture, pour la dernière erreur :

Warning, treated as error:

/home/travis/build/python/python-docs-fr/venv/worktrees/83d3202b92fb4c2fc6df5b035d57f3a1cf715f20/Doc/distutils/setupscript.rst:67:inconsistent term references in translated message. original: [':mod:`distutils`'], translated: []

J'aurais envie de dire que c'est un problème de wrapping car il me semble bien qu'il y a :mod:distutils dans l'original et la traduction.

Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

Relu jusqu'a la ligne 537.

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Relu jusqu'à la ligne 795

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Merci pour ce très gros travail de grande qualité.
Quelques remarques.

@PhilippeGalvan
Copy link
Contributor Author

Merci infiniment pour ta patience et le temps consacré à la relecture!

J'ai fais autant de vérifications que possible de mon coté, il reste seulement une ligne à traduire où je ne pense pas en donner une traduction fidèle.

La CI relève malheureusement encore une erreur en dehors du fichier setupscript.po que je ne sais pas régler


/home/travis/build/python/python-docs-fr/venv/worktrees/83d3202b92fb4c2fc6df5b035d57f3a1cf715f20/Doc/c-api/buffer.rst:92:Error in declarator or parameters

Invalid C declaration: Expected identifier in nested name. [error at 5]

  void \*buf

  -----^

Makefile:49: recipe for target 'build' failed

make[1]: *** [build] Error 2

make[1]: Leaving directory '/home/travis/build/python/python-docs-fr/venv/worktrees/83d3202b92fb4c2fc6df5b035d57f3a1cf715f20/Doc'

Makefile:45: recipe for target 'all' failed

make: *** [all] Error 2

The command "if [ -n "$CHANGED_FILES" ]; then make CPYTHON_PATH=/tmp/cpython/; fi" exited with 2.

christopheNan
christopheNan previously approved these changes Apr 17, 2020
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Merci surtout à toi !
L'erreur Travis provient d'une version de Sphinx qui fait planter la compilation. Cela n'a donc effectivement rien à voir avec ta contribution.
J'essaie de regarder ça ce week-end, mais a priori cela ne doit pas bloquer les commits et la mise en ligne desdits commits.

@PhilippeGalvan PhilippeGalvan dismissed stale reviews from christopheNan via 841abd6 April 17, 2020 18:32
@deronnax
Copy link
Collaborator

deronnax commented May 3, 2020

j'ai mergé le tronc (la 3.8) qui contient des corrections de Christophe pour Sphinx, ça devrait réparer la CI

@deronnax deronnax requested a review from christopheNan May 3, 2020 15:53
@deronnax deronnax dismissed JulienPalard’s stale review May 3, 2020 15:53

Julien pas dispo en ce moment

@deronnax
Copy link
Collaborator

deronnax commented May 3, 2020

Hop, la CI passe. Celle-ci elle est très grosse et en plus c'est une page importante, je dirais que 2 revues seraient pas un luxe. @awecx ?

@deronnax deronnax requested a review from awecx May 3, 2020 15:54
Co-authored-by: Christophe Nanteuil <[email protected]>
christopheNan
christopheNan previously approved these changes May 3, 2020
Co-authored-by: Antoine <[email protected]>
Co-authored-by: Julien Palard <[email protected]>
@deronnax deronnax dismissed stale reviews from christopheNan via eaca5c7 May 5, 2020 10:30
@deronnax
Copy link
Collaborator

deronnax commented May 5, 2020

merci beaucoup awecx pour cette excellente revue (et Christophe aussi d'ailleurs). Il y 2-3 remarques que je challenge

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.

Merci à tous pour cette grosse PR.

@christopheNan christopheNan merged commit 1e1ed82 into python:3.8 May 5, 2020
@christopheNan
Copy link
Contributor

Merci . grosse PR de très bonne qualité.

@deronnax
Copy link
Collaborator

deronnax commented May 5, 2020

Hasta la vista baby. Merci à tous trois pour cette super PR.

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 distutils/setupscript.po
5 participants