-
Notifications
You must be signed in to change notification settings - Fork 266
Traduction de library/ast.po #1691
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 8000 your account
Conversation
Lorsqu'il y a des paragraphes non traduits, c'est soit que je ne comprenais pas complètement, soit qu'il y avait des erreurs, qu'il faudra que j'aille corriger upstream. Je remarque aussi que le fichier source dans le commit du Makefile est assez nettement plus à jour que l'ensemble des sources dans le fichier PO. Il y a eu quelques corrections de fautes de frappe, et surtout l'ajout des nœuds d'AST pour le pattern matching. Est-ce que c'est bien normal? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
|
||
#: library/ast.rst:1357 | ||
msgid "A ``return`` statement." | ||
msgstr "" | ||
msgstr "Instruction ``return``, qui renvoie *value*." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu as remonté upstream ? Si oui, tu peux mettre le ticket en référence ici.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je comptais faire directement une PR pour corriger un certain nombre de petites choses, mais je n'en ai pas encore eu le temps.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@christopheNan Je n'ai pas beaucoup de temps en ce moment pour Python, je préférerais donc fusionner cette PR et remettre la réorganisation de la documentation AST upstream à un peu plus tard si cela ne te dérange pas.
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relecture terminée. Quelques propositions.
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Je mergerai cette PR dès que les tests passeront et qu'elle sera approuvée. DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: automerge) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Ça part en merge, ne touchez à plus rien, je m'en occupe ! DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: automerge_approved_testok) |
Merci beaucoup @jean-abou-samra pour cette PR ! C'est fusionné, suivante ! DisclaimerJe suis un robot fait par l'équipe de l'AFPy et de Traduction I'm a bot made by the Translation and AFPy teams on their free (state: automerge_approved_testok) |
Closes #1167