-
Notifications
You must be signed in to change notification settings - Fork 211
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
[Merged by Bors] - p2p: server: use zap for logging #5979
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5979 +/- ##
=========================================
+ Coverage 80.7% 81.0% +0.2%
=========================================
Files 288 292 +4
Lines 29894 30700 +806
=========================================
+ Hits 24149 24886 +737
- Misses 4158 4178 +20
- Partials 1587 1636 +49 ☔ View full report in Codecov by Sentry. |
Co-authored-by: Matthias Fasching <5011972+fasmat@users.noreply.github.com>
bors merge |
Canceled. |
bors merge |
## Motivation Use `zap` for logging in `p2p/server` instead of the custom logger. One of the minor changes needed for syncv2 Co-authored-by: Ivan Shvedunov <ivan4th@users.noreply.github.com>
Build failed (retrying...): |
## Motivation Use `zap` for logging in `p2p/server` instead of the custom logger. One of the minor changes needed for syncv2 Co-authored-by: Ivan Shvedunov <ivan4th@users.noreply.github.com>
Pull request successfully merged into develop. Build succeeded: |
Motivation
Use
zap
for logging inp2p/server
instead of the custom logger.One of the minor changes needed for syncv2
Description
Minor refactor