-
Notifications
You must be signed in to change notification settings - Fork 266
library/marshal #1478
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
library/marshal #1478
Conversation
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.
Merci pour cette PR Zepmanbc.
Co-authored-by: Antoine <[email protected]>
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.
Deuxième lecture, 2-3 bricoles.
Sinon ça me semble bien.
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.
Au temps pour moi, on parle d'ensembles « figés » et non pas « gelés ».
Co-authored-by: Antoine <[email protected]>
Les tests passent \o/ Bravo et merci @Zepmanbc ! Est-ce qu'un relecteur peut me relire ça ? CC @awecx, @awecx, @awecx (et j'ai rien contre un petit label 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: testok) |
Co-authored-by: Antoine <[email protected]>
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.
Merci pour cette traduction !
Mon grain de sel.
Y'a-il un humain dans le coin pour relire ? CC @awecx, @christopheNan 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) |
(Attention @Zepmanbc j'ai mergé la 3.9 dans ta branche, tu as un merge commit dans ton historique maintenant, ne le rebase pas :p) |
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.
Mes remarques avec un éditeur directement dans github
Co-authored-by: Antoine <[email protected]> Co-authored-by: Christophe Nanteuil <[email protected]>
|
C'est « fichier binaire » dans le glossaire, il faut traduire la balise term par le terme exact sinon ça ne fonctionne pas en effet. |
@JulienPalard la compilation choucroute méchamment. Je l'ai relancée sans succès, tu peux y jeter un œil ? |
library/marshal.po
Outdated
"Lit une valeur du fichier ouvert et la renvoie. Si aucune valeur valide " | ||
"n'est lue (p. ex. parce que les données ont un format décompilé incompatible " | ||
"avec une autre version de Python), :exc:`EOFError`, :exc:`ValueError` ou :" | ||
"exc:`TypeError` est levée. Le fichier doit être un fichier :term:`binaire` " |
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.
Il te manque encore un fichier binaire ici.
Ç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 @Zepmanbc 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) |
Merci 🍰 |
🍻 |
close #1477