-
Notifications
You must be signed in to change notification settings - Fork 25
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
Replace Logger #215
Comments
If I was going to start from scratch then I agree that a logging library would make more sense, but since the logger doesn't change often I don't really see how it would be beneficial |
I understand, it just an enhancement, also I think is a good start to decoupling the dependencies in the project, avoid using a custom logger for something more flexible, Using a custom logging library like Serilog, can enhance the way how the app writes logs, making it async, structured, standard and extensible for future developments |
In order to simplify logger, can be a good enhancement if project use Serilog as base for all logs, all current logger functionality can be covered using serilog enrichments.
Solution: Replace CompilePalLogger with a clasic Serilog Implementation
Alternatives: NLog, Log4Net
The text was updated successfully, but these errors were encountered: