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

go: Remove the integrated ledger support #3128

Merged
merged 1 commit into from
Jul 24, 2020

Conversation

Yawning
Copy link
Contributor

@Yawning Yawning commented Jul 23, 2020

This has been moved to a separate repository.

Note: Yes this updates a bunch of dependencies. The build got super mad when I messed with go.mod.

@Yawning Yawning force-pushed the yawning/feature/die-ledger-die branch 3 times, most recently from 1e4aa66 to 6ef2662 Compare July 23, 2020 15:17
@Yawning Yawning marked this pull request as ready for review July 23, 2020 15:20
@codecov
Copy link

codecov bot commented Jul 23, 2020

Codecov Report

Merging #3128 into master will increase coverage by 0.52%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3128      +/-   ##
==========================================
+ Coverage   68.42%   68.95%   +0.52%     
==========================================
  Files         376      373       -3     
  Lines       37065    36986      -79     
==========================================
+ Hits        25363    25504     +141     
+ Misses       8451     8258     -193     
+ Partials     3251     3224      -27     
Impacted Files Coverage Δ
go/oasis-node/cmd/common/signer/signer.go 57.31% <ø> (+4.62%) ⬆️
go/oasis-node/cmd/signer/signer.go 26.31% <ø> (-12.82%) ⬇️
go/consensus/tendermint/api/errors.go 86.66% <0.00%> (-13.34%) ⬇️
go/worker/compute/executor/committee/state.go 74.07% <0.00%> (-11.12%) ⬇️
...sensus/tendermint/epochtime_mock/epochtime_mock.go 70.63% <0.00%> (-5.56%) ⬇️
go/worker/compute/executor/committee/node.go 61.58% <0.00%> (-4.90%) ⬇️
.../consensus/tendermint/apps/epochtime_mock/state.go 74.46% <0.00%> (-4.26%) ⬇️
go/worker/common/committee/node.go 72.88% <0.00%> (-2.83%) ⬇️
go/worker/compute/merge/committee/node.go 75.54% <0.00%> (-0.73%) ⬇️
go/consensus/tendermint/services.go 82.81% <0.00%> (ø)
... and 32 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 f7a4d38...6602212. Read the comment docs.

go/oasis-node/cmd/signer/signer.go Outdated Show resolved Hide resolved
This has been moved to a separate repository.
@Yawning Yawning force-pushed the yawning/feature/die-ledger-die branch from 6ef2662 to 6602212 Compare July 24, 2020 11:27
@Yawning Yawning merged commit 204b4f7 into master Jul 24, 2020
@Yawning Yawning deleted the yawning/feature/die-ledger-die branch July 24, 2020 11:58
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.

3 participants