-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support custom options at message-builder #63
Conversation
#euvi #eutava |
Por que o coverage caiu tanto? @lucianopf |
Na verdade ele caiu meio porcento, mas não consigo entender pq, dps vou caçar 100% de coverage dnv Eu até testei o fluxo que alterei por completo 😢 |
É que na README.md (da master) ta 89%, ai assustei quando vi o resultado aqui do PR.. mas de boa moço |
Quando for pro ar, só me avisar q eu público no npm. Inclusive, já faz o bump na versão. Acho q pra esse caso, faz sentido ser um |
Consigo tbm publicar, vou fazer então um commit com o bump e quando for publicar de dou um toque, VLW!! 😍 |
Description
This PR aims to update how we merge the logger options with the message options, enabling it to be overwritten by the logger message options.
The main example is enabling the overwrite when logging a specific message setting a custom service name.