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

Problem: block id field in vote could be empty #737

Merged
merged 1 commit into from
Dec 25, 2019

Conversation

yihuang
Copy link
Collaborator

@yihuang yihuang commented Dec 24, 2019

Solution:

  • Fix the problem in tendermint-rs
  • Upgrade dependency

tendermint-rs PR for this: informalsystems/tendermint-rs#103

Solution:
- Fix the problem in tendermint-rs
- upgrade dependency
@leejw51
Copy link
Contributor

leejw51 commented Dec 24, 2019

thanks!
no more sync bug

@codecov
Copy link

codecov bot commented Dec 24, 2019

Codecov Report

Merging #737 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #737      +/-   ##
==========================================
- Coverage   69.69%   69.68%   -0.02%     
==========================================
  Files         131      131              
  Lines       16749    16749              
==========================================
- Hits        11674    11672       -2     
- Misses       5075     5077       +2
Impacted Files Coverage Δ
test-common/src/block_generator.rs 91.89% <100%> (ø) ⬆️
chain-core/src/tx/fee/mod.rs 87.68% <0%> (-0.5%) ⬇️
chain-core/src/common/merkle_tree.rs 98.55% <0%> (-0.25%) ⬇️

Copy link
Contributor

@leejw51 leejw51 left a comment

Choose a reason for hiding this comment

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

good

@devashishdxt
Copy link
Collaborator

bors r+

bors bot added a commit that referenced this pull request Dec 25, 2019
737: Problem: block id field in vote could be empty r=devashishdxt a=yihuang

Solution:
- Fix the problem in tendermint-rs
- Upgrade dependency

tendermint-rs PR for this: informalsystems/tendermint-rs#103

Co-authored-by: yihuang <yi.codeplayer@gmail.com>
@bors
Copy link
Contributor

bors bot commented Dec 25, 2019

@bors bors bot merged commit 3c39821 into crypto-com:master Dec 25, 2019
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.

3 participants