bitcoin icon indicating copy to clipboard operation
bitcoin copied to clipboard
verified publisher icon bitcoin

TxDownloadManager refactor followups

Open glozow opened this issue 1 year ago • 1 comments

Followups to #30110

  • https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1742665719 + a few more doc fixes
  • https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1742884245

glozow avatar Sep 05 '24 12:09 glozow

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process. A summary of reviews will appear here.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #30572 (Halt processing of unrequested transactions v2 by ariard)
  • #30538 (Doc: add a comment referencing past vulnerability next to where it was fixed by darosior)
  • #30116 (p2p: Fill reconciliation sets (Erlay) attempt 2 by sr-gi)
  • #29492 (refactor: Remove redundant definitions by Empact)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

DrahtBot avatar Sep 05 '24 12:09 DrahtBot

closing, squashed into #30110. will reopen if followups need to happen again.

glozow avatar Sep 17 '24 00:09 glozow