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

runtime: Stop using the ring backend for snow #3990

Merged
merged 1 commit into from
Jun 3, 2021

Conversation

kostko
Copy link
Member

@kostko kostko commented Jun 3, 2021

See #2683

@kostko kostko force-pushed the kostko/feature/snow-no-ring branch from 1d19f3a to 87502bb Compare June 3, 2021 07:16
@kostko kostko enabled auto-merge June 3, 2021 07:41
@codecov
Copy link

codecov bot commented Jun 3, 2021

Codecov Report

Merging #3990 (87502bb) into master (aa25d09) will increase coverage by 0.10%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3990      +/-   ##
==========================================
+ Coverage   67.07%   67.18%   +0.10%     
==========================================
  Files         411      411              
  Lines       42082    42082              
==========================================
+ Hits        28228    28272      +44     
+ Misses       9868     9838      -30     
+ Partials     3986     3972      -14     
Impacted Files Coverage Δ
go/consensus/tendermint/abci/state/state.go 54.54% <0.00%> (-9.10%) ⬇️
go/storage/api/root_cache.go 80.00% <0.00%> (-4.45%) ⬇️
go/runtime/host/sandbox/sandbox.go 80.07% <0.00%> (-2.35%) ⬇️
go/runtime/history/history.go 72.04% <0.00%> (-2.16%) ⬇️
go/consensus/tendermint/abci/state.go 70.27% <0.00%> (-2.03%) ⬇️
go/runtime/nodes/versioned.go 73.33% <0.00%> (-1.91%) ⬇️
go/worker/common/p2p/dispatch.go 72.60% <0.00%> (-1.37%) ⬇️
go/storage/mkvs/insert.go 91.78% <0.00%> (-1.37%) ⬇️
go/runtime/client/submitter.go 60.40% <0.00%> (-1.35%) ⬇️
go/registry/api/api.go 53.23% <0.00%> (-1.11%) ⬇️
... and 31 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 aa25d09...87502bb. Read the comment docs.

@kostko kostko merged commit 75dbb6b into master Jun 3, 2021
@kostko kostko deleted the kostko/feature/snow-no-ring branch June 3, 2021 12:22
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