Skip to content
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

feat(logrus): use github.com/hertz-contrib/logger/logrus as logger impl #12

Merged
merged 6 commits into from
Nov 23, 2022

Conversation

CoderPoet
Copy link
Collaborator

What type of PR is this?

feat: use github.com/hertz-contrib/logger/logrus as logger impl

What this PR does / why we need it (English/Chinese):

en: use github.com/hertz-contrib/logger/logrus as logger impl
zh: 使用 github.com/hertz-contrib/logger/logrus 库作为 logger 的实现

Which issue(s) this PR fixes:

@CoderPoet CoderPoet changed the title WIP: feat(logrus): use github.com/hertz-contrib/logger/logrus as logger impl feat(logrus): use github.com/hertz-contrib/logger/logrus as logger impl Nov 9, 2022
@li-jin-gou
Copy link
Contributor

LGTM

@li-jin-gou li-jin-gou merged commit 68d0df9 into hertz-contrib:main Nov 23, 2022
li-jin-gou pushed a commit to li-jin-gou/obs-opentelemetry that referenced this pull request Jan 15, 2024
…on-recording

feat(logging): support non recording span
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.

None yet

3 participants