Replies: 1 comment 2 replies
-
|
I don't like to make these sorts of changes because the tracing crate already provides an interface for configuring this on a per crate basis. It's also a breaking change but one that is potentially less visible and therefore a bigger footgun, so to speak. So without a really compelling reason, I have to decline the offer. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
This tracing doesn't actually provide any meaningful data other than signalling that this method was called (same as the saved user one). Can they be reduced to
debuglevel? I could quickly prepare a PR for that.Since you can’t delete the issue, I’ll repost the reply you deleted, since is clear you prefer to avoid public confrontation and modify comments to make you look better:
Beta Was this translation helpful? Give feedback.
All reactions