dyweb / gommon

A collection of common util libraries for Go
MIT License
27 stars 6 forks source link

[log] Refactor and measure performance using benchmark #90

Closed at15 closed 5 years ago

at15 commented 6 years ago

Trying to fix all those issues https://github.com/dyweb/gommon/issues?q=is%3Aopen+is%3Aissue+label%3Apkg%2Flog

Will first write the doc and list some changes in design, which could be the base of the long pending blog https://github.com/dyweb/blog/issues/28

TODO

Fixed

codecov[bot] commented 6 years ago

Codecov Report

Merging #90 into master will increase coverage by 0.06%. The diff coverage is 30.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
+ Coverage   48.16%   48.22%   +0.06%     
==========================================
  Files          52       56       +4     
  Lines        1796     2030     +234     
==========================================
+ Hits          865      979     +114     
- Misses        847      956     +109     
- Partials       84       95      +11
Impacted Files Coverage Δ
util/cast/cast.go 61.9% <ø> (ø) :arrow_up:
log/generator.go 66.66% <ø> (ø) :arrow_up:
generator/generate.go 0% <ø> (ø) :arrow_up:
generator/shell.go 0% <ø> (ø) :arrow_up:
generator/pkg.go 0% <ø> (ø) :arrow_up:
util/httputil/unix.go 0% <0%> (ø)
log/handlers/cli/handler.go 20% <0%> (+4.81%) :arrow_up:
util/testutil/http.go 0% <0%> (ø) :arrow_up:
log/field.go 75% <100%> (+75%) :arrow_up:
util/runtimeutil/caller.go 92% <100%> (ø) :arrow_up:
... and 20 more

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 ec45062...f1a6678. Read the comment docs.