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

*: regenerate proto #9299

Merged
merged 1 commit into from
Feb 8, 2018
Merged

*: regenerate proto #9299

merged 1 commit into from
Feb 8, 2018

Conversation

hexfusion
Copy link
Contributor

genproto populated a change from 6b775cd breaking it out here in seperate commit.

@yudai

@codecov-io
Copy link

Codecov Report

Merging #9299 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9299      +/-   ##
==========================================
- Coverage   75.55%   75.52%   -0.03%     
==========================================
  Files         365      365              
  Lines       30702    30702              
==========================================
- Hits        23198    23189       -9     
- Misses       5879     5890      +11     
+ Partials     1625     1623       -2
Impacted Files Coverage Δ
clientv3/leasing/util.go 91.66% <0%> (-6.67%) ⬇️
clientv3/leasing/cache.go 87.77% <0%> (-3.89%) ⬇️
rafthttp/msgappv2_codec.go 71.3% <0%> (-1.74%) ⬇️
pkg/schedule/schedule.go 81.69% <0%> (-1.41%) ⬇️
clientv3/op.go 72.56% <0%> (-1.33%) ⬇️
clientv3/leasing/kv.go 90.36% <0%> (-1%) ⬇️
etcdserver/v3_server.go 81.23% <0%> (-0.78%) ⬇️
etcdserver/api/v2http/client.go 90.33% <0%> (-0.45%) ⬇️
clientv3/health_balancer.go 86.28% <0%> (-0.31%) ⬇️
etcdserver/server.go 79.14% <0%> (-0.24%) ⬇️
... and 8 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 a0e17f6...7efe72c. Read the comment docs.

@gyuho gyuho merged commit b309bc6 into etcd-io:master Feb 8, 2018
@yudai
Copy link
Contributor

yudai commented Feb 8, 2018

Thanks!

@yudai
Copy link
Contributor

yudai commented Feb 8, 2018

Oh I see, I missed the -g option. It might be better to switch this option enabled if there is no major drawback.

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