Skip to content

Conversation

achow101
Copy link
Member

No description provided.

@maflcko
Copy link
Member

maflcko commented Dec 14, 2021

code review ACK efa115a and apologies for missing the silent merge conflict in merge commit 9635760

@maflcko
Copy link
Member

maflcko commented Dec 14, 2021

tested that the test now passes

@maflcko maflcko merged commit 9015d11 into bitcoin:master Dec 14, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 14, 2021
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Feb 3, 2022
Summary:
```
This function (check_node_connections) exists in feature_anchors.py and p2p_add_connections.py and does the same thing in both, this PR moves this function to util to avoid repetition and facilitate maintenance.
```

Backport of [[bitcoin/bitcoin#23392 | core#23392]] and [[bitcoin/bitcoin#23774 | core#23774]] (missing import fix).

Depends on D10966.

Ref T1696.

Test Plan:
  ninja check-functional

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

Maniphest Tasks: T1696

Differential Revision: https://reviews.bitcoinabc.org/D10967
@bitcoin bitcoin locked and limited conversation to collaborators Dec 14, 2022
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.

2 participants