Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

add SettableRpcFuture - #205

Merged
pongad merged 2 commits into
googleapis:masterfrom
pongad:settable-future
Feb 16, 2017
Merged

add SettableRpcFuture#205
pongad merged 2 commits into
googleapis:masterfrom
pongad:settable-future

Conversation

@pongad

Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io

codecov-io commented Feb 14, 2017

Copy link
Copy Markdown

Codecov Report

Merging #205 into master will increase coverage by 0.21%.
The diff coverage is 91.66%.

@@ Coverage Diff @@## master #205 +/- ##
============================================
+ Coverage 69.72% 69.93% +0.21% - Complexity 464 475 +11 
============================================
Files 65 66 +1 Lines 2454 2478 +24 Branches 262 262 ============================================
+ Hits 1711 1733 +22 - Misses 646 648 +2 
Partials 97 97
Impacted FilesCoverage ΔComplexity Δ
...ava/com/google/api/gax/core/SettableRpcFuture.java91.66% <91.66%> (ø)11 <11> (?)

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 6947a72...4f2a0a2. Read the comment docs.

@garrettjonesgoogle

Copy link
Copy Markdown
Contributor

Are tests worth it?

@pongad

Copy link
Copy Markdown
ContributorAuthor

@garrettjonesgoogle Not sure if this is an overkill or not. PTAL.

@garrettjonesgoogle

Copy link
Copy Markdown
Contributor

LGTM

@pongad
pongad merged commit 184e5a7 into googleapis:masterFeb 16, 2017
@pongad
pongad deleted the settable-future branch February 16, 2017 20:45
pongad added a commit that referenced this pull request Feb 16, 2017
This should be merged after
#205
so that `SettableRpcFuture` could be used to address
googleapis/google-cloud-java#1613
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@pongad@codecov-io@garrettjonesgoogle