-
Notifications
You must be signed in to change notification settings - Fork 266
Proposition de traduction pour c-api/mapping.po #982
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
Conversation
hey @Aveheuzed ! Je vais relire ca, en attendant, je te conseille de fixer la CI :D |
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.
Écoute c'est pas mal pour moi ! Tu est bien courageux de t'attaquer à la c-api, je l'ai fait aussi (#413) et c'était imbuvable ! tu as choisi une bonne lib !
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.
Quelque erreurs mineures à corriger.
Co-Authored-By: Antoine <[email protected]>
Co-Authored-By: Antoine <[email protected]>
Co-Authored-By: Antoine <[email protected]>
Co-Authored-By: Antoine <[email protected]>
Co-Authored-By: Antoine <[email protected]>
Co-Authored-By: Christophe Nanteuil <[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.
Encore deux ou trois détails et ce sera bon.
Co-Authored-By: Antoine <[email protected]>
@Aveheuzed ping 🏓 |
Alors, finalement, on se décide sur quoi pour :
|
D'après pogrep (
|
correction de la traduction de _slicing_ Co-Authored-By: Antoine <[email protected]>
@Aveheuzed, n'oublie pas de ré-indenter tes modifications avec powrap. |
Travis me sort ça
Autant pour les deux derniers, ok, mais pour le premier point, c'est Travis (enfin, le mec qui l'a configuré) qui a tort, non ? Je vais pas traduire |
Oui la vérification padpo est encore en bêta, pas de problème pour le premier point. Ce n'est d'ailleurs pas pour cela que la construction Travis a échoué, mais pour le problème de l'espace. |
Tous les changements ont bien été pris en compte.
@Aveheuzed, merci pour cette PR 🎆 |
@awecx @Aveheuzed Ce genre de message vient du non respect de :
Cela se passe ici : https://github.com/vpoulailleau/padpo/blob/master/padpo/checkers/glossary.py Ce n'est d'ailleurs qu'un warning et pas une erreur, car il est difficile d'affirmer avec certitude de manière automatique que la traduction est mauvaise (ce qui n'est pas le cas ici par exemple). |
Il y a des choses pas claires dans la V.O., j'ai mis des commentaires pour l'indiquer, mais ça me surpendrait que tout vous aille du premier coup !