You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
When I need to do a wrapper around Zap logger to pass that into 3rd party library for unifying logging, it might come handy to use zap.Log(level, ...) instead of switch or if-else if chain.
However, now I need to do a same wrapper but with Sugared logger. And that doesn't support general Log method.
Adding generic `Log`, `Logf`, `Logw` and `Logln` methods to Sugared
logger.
When I need to do a wrapper around Zap logger to pass that into 3rd
party library for unifying logging, it might come handy to use
zap.Log(level, ...) instead of switch or if-else if chain.
However, now I need to do a same wrapper but with Sugared logger. And
that doesn't support general Log method.
More description in #1405
Is your feature request related to a problem? Please describe.
When I need to do a wrapper around Zap logger to pass that into 3rd party library for unifying logging, it might come handy to use
zap.Log(level, ...)
instead ofswitch
orif-else if
chain.However, now I need to do a same wrapper but with Sugared logger. And that doesn't support general Log method.
Describe the solution you'd like
and the same for
Logf
,Logln
andLogw
.Describe alternatives you've considered
none
Is this a breaking change?
No, this is not breaking change
The text was updated successfully, but these errors were encountered: