👍 brunoerg approved a pull request: "doc: update fuzz instructions when on macOS"
(https://github.com/bitcoin/bitcoin/pull/31954#pullrequestreview-2658264313)
ACK 75486c8ed87a480b9f0c4dc7a10f3cd4eee87b12

I just tested it on macOS 14.3 and worked fine.

💬 mprenditore commented on pull request "Added rescan option for import descriptors":
(https://github.com/bitcoin/bitcoin/pull/31668#issuecomment-2698289660)
> Moved to draft for now. You'll need to squash your commits, and address the various review feedback.

@fanquake code has been updated as per the discussions and all checks are now passing.
Hope now it's good to be merged.

👍 hodlinator approved a pull request: "ci: Test cross-built Windows executables on Windows natively"
(https://github.com/bitcoin/bitcoin/pull/31176#pullrequestreview-2658280804)
re-ACK e3d3500e8088c446c8ab93a67d80ac241a69488c

Range-diff showed no changes beyond rebase since my [previous review](https://github.com/bitcoin/bitcoin/pull/31176#pullrequestreview-2611426260).

Looked through [latest run of PR](https://github.com/bitcoin/bitcoin/actions/runs/13649585472?pr=31176). Suggest some job names that are clearer to me, maybe there's some middle ground, nothing blocking.

💬 hodlinator commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1979832552)
```suggestion
name: 'Linux->Windows cross, no tests'
```

💬 hodlinator commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1979833466)
```suggestion
name: 'Windows, test cross-built'
```

💬 fjahr commented on pull request "test: Use rpc_deprecated only for testing deprecation":
(https://github.com/bitcoin/bitcoin/pull/31977#discussion_r1979855796)
Added, thanks!

💬 fjahr commented on pull request "test: Use rpc_deprecated only for testing deprecation":
(https://github.com/bitcoin/bitcoin/pull/31977#discussion_r1979855960)
fixed

💬 fjahr commented on pull request "test: Use rpc_deprecated only for testing deprecation":
(https://github.com/bitcoin/bitcoin/pull/31977#discussion_r1979856225)
Changed this but with slightly different wording

💬 fjahr commented on pull request "test: Use rpc_deprecated only for testing deprecation":
(https://github.com/bitcoin/bitcoin/pull/31977#issuecomment-2698328931)
Addressed the comments by @rkrux , thanks!

💬 eval-exec commented on pull request "torcontrol: Limit reconnect timeout to max seconds and log delay in whole seconds":
(https://github.com/bitcoin/bitcoin/pull/31979#discussion_r1979862817)
Thank you, updated.

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2698343835)
Addressed @hodlinator's feedback.

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1979867464)
Thanks! [Done](https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2698343835).

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1979867604)
Thanks! [Done](https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2698343835).

📝 mabu44 opened a pull request: "[WIP] refactor: migrate unit tests to Google Test"
(https://github.com/bitcoin/bitcoin/pull/31988)
WIP: both the code and the following motivations will be updated with new information obtained during research and through feedback from reviewers.

#### Motivation
By migrating the unit tests from Boost to Google Test we can increase the chances of removing all dependencies from Boost in the future.

#### Benefits of Google Test
* Most used C++ testing framework with a large community
* Includes Google Mock for creating mock objects
* Similar syntax to Boost, making refactor simpler
*
...

✅ fanquake closed a pull request: "[WIP] refactor: migrate unit tests to Google Test"
(https://github.com/bitcoin/bitcoin/pull/31988)

💬 instagibbs commented on pull request "cluster mempool: introduce TxGraph":
(https://github.com/bitcoin/bitcoin/pull/31363#issuecomment-2698367852)
ACK 72a97c0a07ea6e5a95ab37c8d95e1ea02cff8e92

With the weaker claims in the code re:PostLinearization, it works for me, and obviates the need for an additional harness test.

`git range-diff master d5abb86439e79d4adfbfbd46f833268bbca0bf6e 72a97c0a07ea6e5a95ab37c8d95e1ea02cff8e92`

💬 achow101 commented on pull request "psbt: add non-default sighash types to PSBTs and unify sighash type match checking":
(https://github.com/bitcoin/bitcoin/pull/31622#discussion_r1979899843)
Huh, it does say that. Perhaps I'm remembering an old draft where it was different.

Changed the test to check that it doesn't finalize, and added a commit to implement that behavior.

💬 achow101 commented on pull request "psbt: add non-default sighash types to PSBTs and unify sighash type match checking":
(https://github.com/bitcoin/bitcoin/pull/31622#discussion_r1979900773)
Nothing, it just helps to mutate one which we know is definitely good.

🤔 hebasto reviewed a pull request: "depends: Update libmultiprocess library to fix CI failures"
(https://github.com/bitcoin/bitcoin/pull/31945#pullrequestreview-2658407119)
Post-merge ACK 01f771576608403dbfc5dee4fcdd84ef262f9390.

💬 achow101 commented on pull request "psbt: add non-default sighash types to PSBTs and unify sighash type match checking":
(https://github.com/bitcoin/bitcoin/pull/31622#discussion_r1979937428)
Done