search

pagarme / superbowleto

:football: A microservice to issue, register and manage boletos
MIT License
5 stars 0 forks source link

log: add stack, message and name to error logs #259

Closed grvcoelho closed 6 years ago

grvcoelho commented 6 years ago

Description

On error logs, log the stack, message and name of the errors to improve debuggability.

Your checklist for this pull request

:rotating_light: Please review this items for a good pull request. :four_leaf_clover:

  1. I've read the project's Contributing Guidelines
  2. My commits are well written and follow pagarme/git-style-guide
  3. My changes are well covered by tests and logs
  4. I've updated the project docs (if needed)
  5. I fell safe about this implementation
  6. I feel comfortable with the code I wrote, and I'm not ashamed to show it to my friends

In a good pull request, everything above is true :relaxed:

marquinhoalm commented 6 years ago

Superbowleto senão me engano não tem essa regra @wilkmaia

wilkmaia commented 6 years ago

Deveríamos ter uma regra só para a empresa como um todo.

Se no superbowleto é diferente, temos que chegar num consenso de o que fazer enquanto empresa, grupo de desenvolvimento.

Ter um estilo num lugar e outro em outro é meio awkward :/

grvcoelho commented 6 years ago

@wilkmaia: o superbowleto foi criado antes de criarmos nosso styleguide (https://github.com/pagarme/javascript-style-guide) e é por isso que existem algumas divergências. Não vamos arrumar isso neste PR.