Skip to content

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 <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@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.

Co-Authored-By: Julien Palard <julien@palard.fr>
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 <35002064+christopheNan@users.noreply.github.com>
christopheNan
christopheNan previously approved these changes May 3, 2020
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Co-authored-by: Julien Palard <julien@palard.fr>
@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

deronnax and others added 2 commits May 5, 2020 14:45
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Copy link

@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