Skip to content

feat: add best_block and best_block_tips to dag on status#549

Merged
alexruzenhack merged 6 commits intodevfrom
feat/dag-best-block
Apr 5, 2023
Merged

feat: add best_block and best_block_tips to dag on status#549
alexruzenhack merged 6 commits intodevfrom
feat/dag-best-block

Conversation

@alexruzenhack
Copy link
Copy Markdown
Contributor

@alexruzenhack alexruzenhack commented Apr 3, 2023

Motivation

This addition will allow the hathor-p2p-monitor to monitor synchronization with sync-v2 peers in a simplified way.

Ref.: https://github.com/HathorNetwork/hathor-p2p-monitor/pull/69

Acceptance criteria

  • add best_block and best_block_tips to dag property of /status

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 4, 2023

Codecov Report

Merging #549 (fc94a87) into dev (10e1755) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #549   +/-   ##
=======================================
  Coverage   83.21%   83.21%           
=======================================
  Files         211      211           
  Lines       18659    18665    +6     
  Branches     2592     2593    +1     
=======================================
+ Hits        15527    15532    +5     
  Misses       2563     2563           
- Partials      569      570    +1     
Impacted Files Coverage Δ
hathor/p2p/resources/status.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@alexruzenhack alexruzenhack requested a review from luislhl April 4, 2023 15:09
@alexruzenhack alexruzenhack enabled auto-merge (squash) April 5, 2023 14:11
@alexruzenhack alexruzenhack merged commit b2e18e1 into dev Apr 5, 2023
@alexruzenhack alexruzenhack deleted the feat/dag-best-block branch April 5, 2023 14:13
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.

5 participants