Skip to content

Add Debug to make compatible with log4rs#6

Open
avn-tnl wants to merge 1 commit into
locryus:masterfrom
avn-tnl:master
Open

Add Debug to make compatible with log4rs#6
avn-tnl wants to merge 1 commit into
locryus:masterfrom
avn-tnl:master

Conversation

@avn-tnl

@avn-tnl avn-tnl commented May 6, 2024

Copy link
Copy Markdown

Add an implementation for Debug to make EventLog compatible with Append of log4rs: https://docs.rs/log4rs/latest/log4rs/append/trait.Append.html#impl-Append-for-T
This would allow for EventLog to be used just like the other appenders in log4rs.

@bbqsrc

bbqsrc commented Nov 13, 2024

Copy link
Copy Markdown
Collaborator

@avn-tnl is there a reason for not just returning the inner level directly, or using .debug_tuple ?

@avn-tnl

avn-tnl commented Nov 14, 2024

Copy link
Copy Markdown
Author

@bbqsrc I was not introduced to debug_tuple yet, so sorry about that. I think the type of logger and level is about all the debug information a consumer of the library would want, but I am open to suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants