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

Bug fixes and api updates #48

Merged
merged 12 commits into from
Jun 18, 2019
Merged

Bug fixes and api updates #48

merged 12 commits into from
Jun 18, 2019

Conversation

marcoferrer
Copy link
Owner

No description provided.

@codecov-io
Copy link

codecov-io commented May 20, 2019

Codecov Report

Merging #48 into develop will decrease coverage by 0.05%.
The diff coverage is 60%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop      #48      +/-   ##
=============================================
- Coverage      85.71%   85.66%   -0.06%     
  Complexity        19       19              
=============================================
  Files             15       15              
  Lines            280      279       -1     
  Branches          42       42              
=============================================
- Hits             240      239       -1     
  Misses            13       13              
  Partials          27       27
Impacted Files Coverage Δ Complexity Δ
...utines/call/FlowControlledInboundStreamObserver.kt 75% <ø> (ø) 0 <0> (ø) ⬇️
...oferrer/krotoplus/coroutines/server/ServerCalls.kt 82.5% <0%> (ø) 0 <0> (ø) ⬇️
.../marcoferrer/krotoplus/coroutines/call/CallExts.kt 87.87% <100%> (-0.7%) 0 <0> (ø)
...errer/krotoplus/coroutines/CoroutineCallOptions.kt 100% <100%> (ø) 0 <0> (ø) ⬇️

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 b6b5d94...384098c. Read the comment docs.

@marcoferrer marcoferrer force-pushed the api-updates branch 4 times, most recently from d88eec4 to 6c415ba Compare May 20, 2019 04:05
@marcoferrer marcoferrer merged commit 1b9bd38 into develop Jun 18, 2019
@marcoferrer marcoferrer deleted the api-updates branch June 18, 2019 00:51
marcoferrer added a commit that referenced this pull request Jun 18, 2019
* reduce visibility of FlowControlledInboundStreamObserver to internal

* default server rpc method invocation to CoroutineStart.ATOMIC

* revise coroutine context call option builders

* add tests for atomic server method invocation

* remove message propagation from rpc exception mapper

* add tests for new stub context extensions

* refactor atomic server tests
marcoferrer added a commit that referenced this pull request Jun 18, 2019
* reduce visibility of FlowControlledInboundStreamObserver to internal

* default server rpc method invocation to CoroutineStart.ATOMIC

* revise coroutine context call option builders

* add tests for atomic server method invocation

* remove message propagation from rpc exception mapper

* add tests for new stub context extensions

* refactor atomic server tests
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.

2 participants