Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Problem: v0.5.x client-cli cannot sync (fixes #1552) #1553

Merged
merged 1 commit into from
May 7, 2020

Conversation

tomtau
Copy link
Contributor

@tomtau tomtau commented May 7, 2020

Solution: storage opened only once

@tomtau tomtau force-pushed the fix/client-cli branch from 9ba1761 to c0ec378 Compare May 7, 2020 01:30
@tomtau
Copy link
Contributor Author

tomtau commented May 7, 2020

bors r+

Copy link
Collaborator

@leejw51crypto leejw51crypto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good

@codecov
Copy link

codecov bot commented May 7, 2020

Codecov Report

Merging #1553 into master will increase coverage by 0.36%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1553      +/-   ##
==========================================
+ Coverage   69.98%   70.34%   +0.36%     
==========================================
  Files         150      157       +7     
  Lines       19884    20586     +702     
==========================================
+ Hits        13915    14482     +567     
- Misses       5969     6104     +135     
Impacted Files Coverage Δ
client-cli/src/command.rs 0.00% <0.00%> (ø)
client-rpc/src/rpc/sync_rpc.rs 0.00% <0.00%> (ø)
client-rpc/src/rpc/transaction_rpc.rs 85.41% <0.00%> (ø)
client-rpc/src/lib.rs 50.00% <0.00%> (ø)
client-rpc/src/rpc/sync_worker.rs 0.00% <0.00%> (ø)
client-rpc/src/rpc/multisig_rpc.rs 60.71% <0.00%> (ø)
client-rpc/src/rpc/wallet_rpc.rs 80.15% <0.00%> (ø)
client-rpc/src/rpc/staking_rpc.rs 0.00% <0.00%> (ø)
client-common/src/error.rs 66.19% <0.00%> (+1.40%) ⬆️
client-core/src/mnemonic.rs 93.75% <0.00%> (+2.08%) ⬆️
... and 5 more

@bors
Copy link
Contributor

bors bot commented May 7, 2020

Build succeeded:

@bors bors bot merged commit 3fd0856 into crypto-com:master May 7, 2020
Sign up for free to 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.

5 participants