Skip to content

Traduction de library/argparse.po, suite #1412

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 40 commits into from
Sep 30, 2020

Conversation

ygingras
Copy link
Contributor

No description provided.

@ygingras
Copy link
Contributor Author

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

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.

Sacré boulot, bravo :)

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

@JulienPalard
Copy link
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
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
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
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.

Copy link
Contributor

@PhilippeGalvan PhilippeGalvan left a comment

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 ;) !

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>
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>
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