Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Library.logging.handlers.po #1423

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 25 commits into from
Dec 16, 2020

Conversation

PhilippeGalvan
Copy link
Contributor

@PhilippeGalvan PhilippeGalvan commented Sep 24, 2020

First half for library/logging.handlers.po.

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci beaucoup pour cette très riche contribution.
Je pense qu'il faudrait harmoniser entre « roulement » et « rotation ».
Enfin, elle est assez longue comme ceci. Si tu continues la traduction de la page, merci d'ouvrir une autre PR.

Co-authored-by: Christophe Nanteuil <[email protected]>
@PhilippeGalvan
Copy link
Contributor Author

Merci pour la revue, pas mal de traductions littérales, aurais-tu un conseil sur le degré de "littéralité" de la traduction entre coller au phrasé de l'auteur vs coller à du français plus clair et plus agréable à lire ?

Souhaites-tu que je refasses une passe pour harmoniser rotation/roulement ?

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est au traducteur de voir jusqu'où aller pour être le plus idiomatique possible, et en tous cas être meilleur que Google Translate ou Deepl (même s'ils sont utiles pour accélérer le processus).

Je pense que ça vaudrait le coup de voir ce que ça donne avec roulement uniquement (qui me paraît plus idiomatique).

@JulienPalard
Copy link
Member

@PhilippeGalvan Sais-tu si tu pourras regarder ça prochainement ? Tout va bien ? :)

@PhilippeGalvan
Copy link
Contributor Author

@JulienPalard effectivement, longue absence, je vais essayer de conclure ça d'ici le weekend prochain pour que ça ne traine plus :)

@JulienPalard
Copy link
Member

JulienPalard commented Nov 30, 2020

@JulienPalard effectivement, longue absence

Aucun souci !

je vais essayer de conclure ça d'ici le weekend prochain pour que ça ne traine plus :)

Avec plaisir, mais c'est le monde de l'open source ici, pas de la SSII :) y'a pas de deadlines : c'est pas la pein de t'en mettre ;)

(moi quand je dis « je m'y met le weekend prochain », j'ai un truc de dernier moment ce weekend là…)

Je m'inquiétais juste de savoir si tout va bien :) (en en profitant pour te remettre le sujet en tête au cas où :p)

@PhilippeGalvan
Copy link
Contributor Author

PhilippeGalvan commented Dec 7, 2020

@JulienPalard effectivement, longue absence

Aucun souci !

je vais essayer de conclure ça d'ici le weekend prochain pour que ça ne traine plus :)

Avec plaisir, mais c'est le monde de l'open source ici, pas de la SSII :) y'a pas de deadlines : c'est pas la pein de t'en mettre ;)

(moi quand je dis « je m'y met le weekend prochain », j'ai un truc de dernier moment ce weekend là…)

Je m'inquiétais juste de savoir si tout va bien :) (en en profitant pour te remettre le sujet en tête au cas où :p)

Ah ah c'est drole j'ai justement eu un truc ce week end là ;) !
Effectivement j'ai pas de pression "pro" mais c'est bien de me relancer quand ça fait 2 à 3 mois quand même, j'apprécie la prise de signe de vie :p!

Ces derniers temps j'ai laissé un peu trop de sujet en suspens ^^', c'est bien de terminer ce qui est entamé ;) !

@JulienPalard JulienPalard added the 🤖 automerge The PR will be auto merged label Dec 8, 2020
@PyDocTeur
Copy link

Y'a-il un humain dans le coin pour relire ? CC @christopheNan


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: automerge)
PyDocTeur v1.8.6

@@ -340,26 +451,36 @@ msgid ""
"exception during an :meth:`emit` call, i.e. via the :meth:`handleError` "
"method of the handler."
msgstr ""
"La raison d’être de ces attributs est de vous épargner la création d’une "
"sous-classe — vous pouvez utiliser les mêmes appels pour des instances de :"
"class:`RotatingFileHandler` et :class:`TimedRotatingFileHandler`—. Si le "
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"class:`RotatingFileHandler` et :class:`TimedRotatingFileHandler`. Si le "
"class:`RotatingFileHandler` et :class:`TimedRotatingFileHandler`. Si le "

JulienPalard
JulienPalard previously approved these changes Dec 8, 2020
@JulienPalard
Copy link
Member

Désolé, j'ai review, puis j'ai poussé, github a cru que le fait que je pousse invalidait ma review, mais non :p

@PhilippeGalvan
Copy link
Contributor Author

Mais du coup c'est toujours pas bon :p! Y a une review de ChristopheNan qui passe pas pourtant tout est marqué en "resolved"!

@JulienPalard
Copy link
Member

Trouvé quelques petits coquilles en relisant, sinon c'est bon pour moi.

@PhilippeGalvan
Copy link
Contributor Author

Merci pour la relecture et toutes ces corrections !

@JulienPalard
Copy link
Member

@christopheNan C'est bon pour toi ?

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci beaucoup pour cette contribution.
4 bricoles.

Co-authored-by: Christophe Nanteuil <[email protected]>
christopheNan
christopheNan previously approved these changes Dec 16, 2020
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 🚀

@JulienPalard JulienPalard merged commit 92afc86 into python:3.9 Dec 16, 2020
@JulienPalard
Copy link
Member

Belle PR merci @PhilippeGalvan !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge The PR will be auto merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants