Skip to content

Traduction de library/argparse.po, suite#1412

Merged
JulienPalard merged 40 commits into
python:3.9from
ygingras:library-argparse-p2
Sep 30, 2020
Merged

Traduction de library/argparse.po, suite#1412
JulienPalard merged 40 commits into
python:3.9from
ygingras:library-argparse-p2

Conversation

@ygingras

Copy link
Copy Markdown
Contributor

No description provided.

@ygingras

Copy link
Copy Markdown
Contributor Author

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

Comment thread library/argparse.po

@JulienPalard JulienPalard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sacré boulot, bravo :)

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

Comment thread library/argparse.po Outdated
Comment thread library/argparse.po
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
@JulienPalard

Copy link
Copy Markdown
Member

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
Copy Markdown
Contributor Author

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.

@ygingras

Copy link
Copy Markdown
Contributor Author

Je n'ai rien d'autre à changer pour cette PR. J'en ai une autre de prête à partir avec le reste des messages dès que celle ci est fusionnée.

@ygingras

Copy link
Copy Markdown
Contributor Author

@JulienPalard, je vais avoir besoin de ton sceau d'approbation pour débloquer cette PR. Tous les commentaires on été pris en compte, mais Github la garde bloquée.

@PhilippeGalvan PhilippeGalvan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ygingras. Effectivement sacré boulot!
C'est ma première revue de code sur ce projet, ça vaut le coup de prendre mes propositions pour ce qu'elles sont mais c'est principalement des petites typos ou des tournures de phrase différentes.

Comme on en a parlé hier, parfois il faut aller voir ce qu'il y a sous le capot donc en fin de compte je pense que tu es mieux placé pour l'interprétation de ce que font effectivement les méthodes ;) !

Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
ygingras and others added 2 commits September 22, 2020 20:59
Co-authored-by: PhilippeGalvan <49376995+PhilippeGalvan@users.noreply.github.com>
Co-authored-by: PhilippeGalvan <49376995+PhilippeGalvan@users.noreply.github.com>
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
Comment thread library/argparse.po Outdated
ygingras and others added 4 commits September 24, 2020 15:58
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: PhilippeGalvan <49376995+PhilippeGalvan@users.noreply.github.com>
Comment thread library/argparse.po Outdated
ygingras and others added 2 commits September 25, 2020 08:14
Co-authored-by: Vincent Poulailleau <vpoulailleau@gmail.com>
@JulienPalard JulienPalard dismissed christopheNan’s stale review September 30, 2020 14:40

J'ai vérifié, tout est correctement pris en compte.

@JulienPalard JulienPalard merged commit 98137cf into python:3.9 Sep 30, 2020
@ygingras ygingras deleted the library-argparse-p2 branch February 9, 2021 18:49
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.

5 participants