-
Notifications
You must be signed in to change notification settings - Fork 3
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
[TECH] Refactorer le logger #254
Conversation
Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr254.review.pix.fr |
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.
C'est beaucoup plus clair !
9f63a1e
to
937e38a
Compare
937e38a
to
5bb0e29
Compare
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.
J'ai fait le rebase sur main, prêt à être mergé
6df9963
to
b78e110
Compare
event, | ||
message: typeof message === 'object' ? JSON.stringify(message) : message, | ||
job, | ||
stack: typeof stack === 'object' ? JSON.stringify(stack) : stack, |
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.
Marrant parce qu'on avait fait ça avant :
stack = JSON.stringify(message);
🦄 Problème
Le logger utilise
En plus, on logge le message au lieu de la stack.
🤖 Proposition
Créer un nouvel objet.
Nommer le paramètre.
Logger le message au lieu de la stack.
💯 Pour tester
Vérifier la CI