Skip to content

add support for client unary interceptors in pepclient#247

Open
drewwells wants to merge 2 commits intomasterfrom
feature/unaryClientInterceptors
Open

add support for client unary interceptors in pepclient#247
drewwells wants to merge 2 commits intomasterfrom
feature/unaryClientInterceptors

Conversation

@drewwells
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Mar 28, 2020

Codecov Report

Merging #247 into master will increase coverage by 0.01%.
The diff coverage is 43.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
+ Coverage   73.79%   73.81%   +0.01%     
==========================================
  Files         164      164              
  Lines       15503    15511       +8     
==========================================
+ Hits        11440    11449       +9     
+ Misses       3238     3237       -1     
  Partials      825      825              
Impacted Files Coverage Δ
pep/unary_client.go 72.07% <30.76%> (+0.37%) ⬆️
pep/client.go 100.00% <100.00%> (ø)
pep/streaming_connection.go 82.89% <0.00%> (+0.65%) ⬆️

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 2ae78bb...747bac5. Read the comment docs.

@drewwells
Copy link
Contributor Author

This passes tests, why is Travis not building the PR?

PASS
coverage: 89.7% of statements
ok      github.com/infobloxopen/themis/contrib/coredns/policy   4.338s

@vasili-v vasili-v removed their request for review April 3, 2020 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant