This allows to remove code.
Also, required for https://github.com/bitcoin/bitcoin/pull/18470
test: Use ConnmanTestMsg from test lib in denialofservice_tests #22203
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2106-testConnman changing 1 files +14 −31-
MarcoFalke commented at 7:32 PM on June 9, 2021: member
-
test: Use ConnmanTestMsg from test lib in denialofservice_tests fa72fce7c9
-
practicalswift commented at 8:03 PM on June 9, 2021: contributor
Concept ACK: +14 -31 LOC is nice
- DrahtBot added the label Tests on Jun 9, 2021
-
DrahtBot commented at 11:54 PM on June 9, 2021: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #20966 (banman: save the banlist in a JSON format on disk by vasild)
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 cross-referenced this on Jun 10, 2021 from issue banman: save the banlist in a JSON format on disk by vasild
- fanquake approved
-
fanquake commented at 12:53 PM on June 10, 2021: member
ACK fa72fce7c948185752a01002000ea511809146ed
-
mjdietzx commented at 2:41 PM on June 10, 2021: contributor
crACK fa72fce7c948185752a01002000ea511809146ed 👍👍
- fanquake merged this on Jun 11, 2021
- fanquake closed this on Jun 11, 2021
- MarcoFalke deleted the branch on Jun 11, 2021
- sidhujag referenced this in commit 5d4e00cdc3 on Jun 11, 2021
- gwillen referenced this in commit e39e6e320d on Jun 1, 2022
- bitcoin locked this on Aug 16, 2022
Contributors
Labels