Skip to content

rpc/client: take context as first param#5347

Merged
melekes merged 8 commits intomasterfrom
anton/api-should-take-context-5145
Sep 23, 2020
Merged

rpc/client: take context as first param#5347
melekes merged 8 commits intomasterfrom
anton/api-should-take-context-5145

Conversation

@melekes
Copy link
Copy Markdown
Contributor

@melekes melekes commented Sep 9, 2020

Closes #5145

@melekes melekes force-pushed the anton/api-should-take-context-5145 branch from 1d757a8 to b47f4a1 Compare September 9, 2020 13:07
@melekes melekes self-assigned this Sep 11, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 11, 2020

Codecov Report

Merging #5347 into master will increase coverage by 0.59%.
The diff coverage is 41.15%.

@@            Coverage Diff             @@
##           master    #5347      +/-   ##
==========================================
+ Coverage   60.83%   61.43%   +0.59%     
==========================================
  Files         203      259      +56     
  Lines       20526    23378    +2852     
==========================================
+ Hits        12488    14363    +1875     
- Misses       6961     7564     +603     
- Partials     1077     1451     +374     
Impacted Files Coverage Δ
abci/client/grpc_client.go 0.00% <0.00%> (ø)
abci/example/kvstore/persistent_kvstore.go 40.49% <0.00%> (-5.22%) ⬇️
blockchain/v0/pool.go 76.01% <0.00%> (-2.65%) ⬇️
blockchain/v2/processor_context.go 68.00% <0.00%> (+4.84%) ⬆️
cmd/tendermint/commands/lite.go 19.16% <0.00%> (-0.35%) ⬇️
cmd/tendermint/commands/reset_priv_validator.go 3.44% <0.00%> (-2.44%) ⬇️
cmd/tendermint/commands/run_node.go 0.00% <0.00%> (ø)
config/toml.go 60.97% <ø> (-4.99%) ⬇️
consensus/replay_file.go 0.00% <0.00%> (ø)
consensus/replay_stubs.go 57.57% <ø> (-3.30%) ⬇️
... and 289 more

@melekes melekes force-pushed the anton/api-should-take-context-5145 branch from 0f84dc2 to af3bbae Compare September 11, 2020 11:35
@melekes melekes force-pushed the anton/api-should-take-context-5145 branch from af3bbae to 3e53dd6 Compare September 11, 2020 11:41
@melekes melekes marked this pull request as ready for review September 11, 2020 11:57
@melekes melekes added the T:breaking Type: Breaking Change label Sep 14, 2020
Comment thread light/client_test.go Outdated
Comment thread rpc/client/http/http.go Outdated
Comment thread statesync/snapshots.go Outdated
@melekes melekes force-pushed the anton/api-should-take-context-5145 branch from c0330fd to 0bb759d Compare September 22, 2020 15:25
@melekes melekes merged commit 85a4be8 into master Sep 23, 2020
@melekes melekes deleted the anton/api-should-take-context-5145 branch September 23, 2020 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:breaking Type: Breaking Change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

light: API should take a context

2 participants