rust-bitcoin
/
rust-bitcoin
/
15762947276
83%
|
Ran
|
Jobs
1
|
Files
130
|
Run time
1min
|
Badge
Embed ▾
README BADGES
|
coverage: 84.386% (-0.005%) from 84.391%
push
github
apoelstraMerge rust-bitcoin/rust-bitcoin#4589: Move `bitcoin::p2p` into `p2p` <a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/d9cf7270eb457fd660fa505701895ab4756e394d">d9cf7270e Move `bitcoin/p2p` into `p2p` (rustaceanrob) Pull request description: Could use a rebase on #4568 Somehow lost the branch of my original draft so some of the review context is removed, but all comments were addressed in separate PRs. First commit moves everything required to implement encoding within `p2p`, so we have a reference for functions to add to future moves to `internals`. Second commit does the relocation and third commit removes the `bitcoin/p2p` module. ACKs for top commit: apoelstra: ACK d9cf7270eb457fd660fa505701895ab4756e394d; successfully ran local tests tcharding: ACK d9cf7270eb457fd660fa505701895ab4756e394d Tree-SHA512: 31bf96078e61183767a00e4d1d2e3b2e5b7
206 of 258 new or added lines in 7 files covered. (79.84%)
23624 of 27995 relevant lines covered (84.39%)
15829.05 hits per line
Uncovered Changes
| Lines | Coverage | ∆ | File |
|---|---|---|---|
| 41 |
78.42 |
p2p/src/lib.rs | |
| 7 |
86.27 |
p2p/src/consensus.rs | |
| 2 |
67.06 |
p2p/src/message_network.rs | |
| 1 |
92.61 |
p2p/src/message.rs | |
| 1 |
60.0 |
p2p/src/message_bloom.rs |
Jobs
| ID | Job ID | Ran | Files | Coverage | |
|---|---|---|---|---|---|
| 1 | 15762947276.1 | 130 |
84.39 |
GitHub Action Run |
