Skip to content
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

Utilise __name__ pour le nom des loggers #4724

Merged
merged 1 commit into from
Oct 16, 2017

Conversation

motet-a
Copy link
Contributor

@motet-a motet-a commented Oct 14, 2017

C’est plus verbeux, mais c’est plus uniforme. Je pense que c’est bénéfique.

Notez que certains loggers propres à une classe s’appellent
zds.nom.du.module.NomDeLaClasse.

Contrôle qualité

Regardez le diff.

@motet-a motet-a added the C-Back Concerne le back-end Django label Oct 14, 2017
@motet-a motet-a changed the title Utiliser __name__ pour le nom des loggers Utilise __name__ pour le nom des loggers Oct 14, 2017
C’est plus verbeux, mais c’est plus uniforme. Je pense que c’est
bénéfique.

Notez que certains loggers propres à une classe s’appellent
'zds.nom.du.module.NomDeLaClasse'.
@coveralls
Copy link

coveralls commented Oct 14, 2017

Coverage Status

Coverage increased (+0.0006%) to 89.731% when pulling 7a0d0fe on motet-a:get-logger-name into 1571589 on zestedesavoir:dev.

@zestedesavoir zestedesavoir deleted a comment from coveralls Oct 14, 2017
@motet-a motet-a added the QA svp label Oct 15, 2017
@artragis artragis merged commit 9a583ed into zestedesavoir:dev Oct 16, 2017
@motet-a motet-a added this to the Version de développement milestone Oct 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants