Skip to content

Translate simple_stmts.po #1097

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 6 commits into from
Dec 19, 2019
Merged

Translate simple_stmts.po #1097

merged 6 commits into from
Dec 19, 2019

Conversation

AntoineVenier
Copy link
Contributor

Fix #1087
First pull request

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 pour cette première PR.
Je t'ai fait quelques remarques sur les énumérations notamment. Malheureusement, Github ne permet pas de suggérer une modification n'importe où dans un fichier, mais seulement à proximité des lignes que tu as modifiées ; je n'ai pas toujours pu directement te proposer une correction.
N'oublie pas de réindenter (avec powrap) le fichier une fois ces remarques prises en compte.

@awecx
Copy link

awecx commented Dec 17, 2019

Ping :-)

@AntoineVenier
Copy link
Contributor Author

Bonjour Antoine,
Merci pour toutes ces remarques et désolé pour le temps de réponse je m'occupe de ça cette semaine.

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 pour les corrections. Au passage, s'il tu acceptes une suggestion sans discussion, tu peux toi-même résoudre la conversation.
Encore 2 ou 3 points et ça sera bon.
Merci.

@awecx
Copy link

awecx commented Dec 18, 2019

Merci pour ces corrections.
Par contre le changement sur le tiret a dû casser l'indentation ; il faut (de nouveau) réindenter le fichier.
Dans ta branche :
git pull pour récupérer les modifs' depuis Github
powrap reference/simple_stmts.po pour reformater
Tu repush et on pourra merger ta PR.

@AntoineVenier
Copy link
Contributor Author

Merci beaucoup pour toutes ces explications .

@awecx awecx merged commit 51fd39c into python:3.8 Dec 19, 2019
@awecx
Copy link

awecx commented Dec 19, 2019

Merci pour cette PR ; la documentation officielle sera mise à jour dans quelques heures.
N'hésite pas à commencer d'autres traductions ou à reviewer des pull requests si le cœur t'en dit.

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 reference/simple_stmts.po
2 participants