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

Traduction de library/argparse.po, suite #1412

Open
wants to merge 30 commits into
base: 3.9
from

Conversation

@ygingras
Copy link
Contributor

ygingras commented Sep 12, 2020

No description provided.

ygingras added 27 commits Sep 7, 2020
@ygingras
Copy link
Contributor Author

ygingras commented Sep 12, 2020

Je m'occupe de résoudre les conflits avec la PR précédente dès qu'elle est intégrée.

library/argparse.po Outdated Show resolved Hide resolved
Copy link
Member

JulienPalard left a comment

Sacré boulot, bravo :)

J'ai mis quelques remarques, mais globalement c'est bon pour moi.

library/argparse.po Outdated Show resolved Hide resolved
library/argparse.po Show resolved Hide resolved

#: library/argparse.rst:817
msgid "Note, the *default* will be ``None`` unless explicitly set to *0*."
msgstr ""
"Prenez note que la valeur par défaut sera ``None`` à moins qu'elle soit "

This comment has been minimized.

@JulienPalard

JulienPalard Sep 13, 2020

Member

Ici, default est le nom du paramètre pour la valeur par défaut, il serait bon de le garder orthographié default, en italique.

library/argparse.po Outdated Show resolved Hide resolved
library/argparse.po Outdated Show resolved Hide resolved
library/argparse.po Outdated Show resolved Hide resolved
library/argparse.po Outdated Show resolved Hide resolved
@@ -1039,18 +1400,29 @@ msgid ""
"arguments to :meth:`~ArgumentParser.add_argument`, e.g. ``%(default)s``, ``"
"%(type)s``, etc.::"
msgstr ""
"La chaîne ``help`` peut contenir des spécificateurs de formatage afin "

This comment has been minimized.

@JulienPalard

JulienPalard Sep 13, 2020

Member

Dans ce paragraphe, sphinx se plaint de l'absence de default_, pourrais-tu tenter de l'insérer ?

library/argparse.po Outdated Show resolved Hide resolved
library/argparse.po Outdated Show resolved Hide resolved
ygingras and others added 3 commits Sep 13, 2020
Co-authored-by: Julien Palard <[email protected]>
@JulienPalard
Copy link
Member

JulienPalard commented Sep 14, 2020

LGTM, je te laisse passer un coup de powrap, ou de poedit pour rewrapper le fichier à la norme. (Le souci de wrapping vient typiquement des suggestions github, qui sont faites "sur la ligne", sans rewrapper, dit autremment c'est donc ma faute.).

@ygingras
Copy link
Contributor Author

ygingras commented Sep 14, 2020

Je fais une passe de relecture avec le groupe de traduction de Montréal-Python ce soir, puis je pousse un powrap après toutes les révisions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.