Skip to content

Conversation

@glyg
Copy link

@glyg glyg commented Oct 9, 2018

I'm about a third through the advanced tutorial

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 de bien distinguer le concept de journalisation (à écrire en français dans la page) du module logging qui implémente cette journalisation.
Le français utilise plutôt des substantifs dans les titres alors que l'anglais utilise le gérondif.
La grande majorité des futurs peuvent être passés au présent pour être plus concis et plus clair en français.

@christopheNan
Copy link
Contributor

pour la mise en forme, il convient de mettre tous les termes non français (termes anglais ou de code Python) entre astérisques ou entre accents graves (backtick en anglais).

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.

Quelques problèmes à corriger.

awecx and others added 9 commits November 12, 2018 10:51
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
awecx and others added 4 commits November 12, 2018 10:56
Co-Authored-By: glyg <guillaume@damcb.com>
Co-Authored-By: glyg <guillaume@damcb.com>
@JulienPalard
Copy link
Member

Tu légèrement cassé les entêtes (tu as enlevé la Language-Team par exemple) regard le diff.

Tu as rajouté (automatiquement probablement) une mention érronée sur la redistribution, toujours dans les entêtes.

Et les féminins a la place des neutres c'est volontaire ?

@JulienPalard
Copy link
Member

J'ai fait un petit push pour fixer la CI, c'était un souci d'espaces après les "::". (il suffit de " : ::", mais en rajouter après les "::" fait perdre le parseur rst).

Pas de merge tant qu'il y a un WIP dans le sujet, lorsque pour toi la PR est prête n'oublie pas de l'enlever.

@glyg
Copy link
Author

glyg commented Nov 13, 2018

S'il vaut mieux fermer cette PR et reprendre sur une nouvelle la continuation de la traduction du howto - il reste pas mal de taf dans la partie advanced - pour moi c'est bon.

@JulienPalard JulienPalard merged commit 464086a into python:3.7 Nov 13, 2018
@JulienPalard
Copy link
Member

BOOM 34% \o/

@glyg glyg deleted the logging branch December 6, 2018 20:05
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.

6 participants