Skip to content

Comments

Add github.com/blendle/zapdriver to list of extensions#596

Merged
akshayjshah merged 1 commit intouber-go:masterfrom
JeanMertz:patch-1
May 30, 2018
Merged

Add github.com/blendle/zapdriver to list of extensions#596
akshayjshah merged 1 commit intouber-go:masterfrom
JeanMertz:patch-1

Conversation

@JeanMertz
Copy link
Contributor

See: #452 (comment)

For those interested, we've built a small Zap-based Stackdriver library that handles most of the heavy lifting of getting Stackdriver-formatted logs:

https://github.com/blendle/zapdriver

It produces "LogEntry v2" compatible logs to which @piotrkowalczuk pointed. It also includes some helper functions that produce the appropriate fields, such as HTTP, SourceLocation, Label, and Operation.

Feel free to take it for a spin and report any issues you find. I'm sure there is still some functionality missing, but it did get us going.

@CLAassistant
Copy link

CLAassistant commented May 27, 2018

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented May 27, 2018

Codecov Report

Merging #596 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #596   +/-   ##
=======================================
  Coverage   97.47%   97.47%           
=======================================
  Files          39       39           
  Lines        2017     2017           
=======================================
  Hits         1966     1966           
  Misses         43       43           
  Partials        8        8

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2dc8d10...c173905. Read the comment docs.

@akshayjshah
Copy link
Contributor

Thanks for the addition, @JeanMertz!

@akshayjshah akshayjshah merged commit ceb747c into uber-go:master May 30, 2018
@JeanMertz JeanMertz deleted the patch-1 branch May 30, 2018 22:12
RenovZ pushed a commit to RenovZ/zap that referenced this pull request Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants