Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Small optimization to avoid unnecesarry allocations. #228

Merged
merged 2 commits into from
Oct 5, 2019

Conversation

bogdandrutu
Copy link
Contributor

No description provided.

@bogdandrutu bogdandrutu requested review from rghetia and songy23 and removed request for rghetia October 5, 2019 01:27
@codecov-io
Copy link

Codecov Report

Merging #228 into master will decrease coverage by 0.02%.
The diff coverage is 60%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #228      +/-   ##
==========================================
- Coverage   73.39%   73.37%   -0.03%     
==========================================
  Files          15       15              
  Lines        1609     1615       +6     
==========================================
+ Hits         1181     1185       +4     
- Misses        351      352       +1     
- Partials       77       78       +1
Impacted Files Coverage Δ
stats.go 78.46% <100%> (+0.12%) ⬆️
metrics.go 78.94% <50%> (+0.15%) ⬆️
metrics_proto.go 88.09% <57.14%> (-0.53%) ⬇️

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 9cc5395...a216577. Read the comment docs.

Copy link
Contributor

@songy23 songy23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants