📝 nowyouseeithaha opened a pull request: "Patch-1"
(https://github.com/bitcoin/bitcoin/pull/27764)

✅ pinheadmz closed a pull request: "Patch-1"
(https://github.com/bitcoin/bitcoin/pull/27764)

📝 fanquake locked a pull request: "Patch-1"
(https://github.com/bitcoin/bitcoin/pull/27764)

💬 ajtowns commented on pull request "p2p: Log addresses of stalling peers":
(https://github.com/bitcoin/bitcoin/pull/27761#discussion_r1206228638)
Should be `Peer=%d%s` (the result of strprintf isn't a number) and `peeraddr=%s` (for better consistency with other `fLogIPs` logs)

💬 mzumsande commented on pull request "p2p: Log addresses of stalling peers":
(https://github.com/bitcoin/bitcoin/pull/27761#discussion_r1206242431)
fixed, thanks.

💬 MarcoFalke commented on pull request "kernel: Remove shutdown from kernel library":
(https://github.com/bitcoin/bitcoin/pull/27711#issuecomment-1563846514)
> It also provides an easy way to see when calls to shutdown are made without returning to the calling function.

Can you explain this? Am I correct when guessing that any added EXIT* macro, and not the BUBBLE* macros are indicating a missing return?

📝 MarcoFalke opened a pull request: "test: Throw error when -signetchallenge is non-hex"
(https://github.com/bitcoin/bitcoin/pull/27765)
Instead of silently parsing non-hex to an empty challenge, throw an error.

Also, add missing includes while touching the file.

💬 MarcoFalke commented on pull request "test: Move test_chain_listunspent wallet check from mempool_packages to wallet_basic":
(https://github.com/bitcoin/bitcoin/pull/27735#issuecomment-1563922719)
Thx, done

💬 MarcoFalke commented on pull request "doc: Add doc/release-notes/release-notes-25.0.md":
(https://github.com/bitcoin/bitcoin/pull/27751#discussion_r1206361499)
```suggestion
- Transactions of non-witness size 65 bytes and above are now allowed by mempool
```

💬 MarcoFalke commented on pull request "doc: Add doc/release-notes/release-notes-25.0.md":
(https://github.com/bitcoin/bitcoin/pull/27751#discussion_r1206363521)
why? There is no list of all pulls in the notes, so the number is without context?

💬 MarcoFalke commented on pull request "doc: Add doc/release-notes/release-notes-25.0.md":
(https://github.com/bitcoin/bitcoin/pull/27751#issuecomment-1563956991)
Fixed the broken link. Not sure about the other nits.

💬 stratospher commented on pull request "p2p: Log addresses of stalling peers":
(https://github.com/bitcoin/bitcoin/pull/27761#issuecomment-1563972944)
tACK fb02a3c.

nice to have this! cross checked locations where we'd like inbound peer/disconnection peeraddr logging.
(you may have to rebase on master for green CI.)

👍 TheCharlatan approved a pull request: "init: Error if ignored bitcoin.conf file is found"
(https://github.com/bitcoin/bitcoin/pull/27302#pullrequestreview-1445624581)
ACK eefe56967b4eb4b5144325cde4f40fc1cbde3e65

💬 1ma commented on pull request "Allow accepting non-standard transactions on mainnet":
(https://github.com/bitcoin/bitcoin/pull/27578#issuecomment-1564059817)
There are no risks in submitting non-standard transactions to your own node's mempool. Its direct peers will simply discard these when your node broadcasts them, and they will eventually be purged from your mempool as any other non-confirmed transaction.

If anything, node runners want to be able to apply more filters/standardness rules (increased sovereignty), not disable them (decreased sovereignty).

👍 TheCharlatan approved a pull request: "test: Throw error when -signetchallenge is non-hex"
(https://github.com/bitcoin/bitcoin/pull/27765#pullrequestreview-1445755490)
Nice, ACK fa6b11a55663e70369bfbbba5fccc55b33f2b310

💬 kevkevinpal commented on pull request "test: Throw error when -signetchallenge is non-hex":
(https://github.com/bitcoin/bitcoin/pull/27765#issuecomment-1564100105)
ACK [fa6b11a](https://github.com/bitcoin/bitcoin/pull/27765/commits/fa6b11a55663e70369bfbbba5fccc55b33f2b310)

💬 TheCharlatan commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#issuecomment-1564122573)
Still not sure what is going on here:

```
MAKEJOBS="-j12" FILE_ENV="./ci/test/00_setup_env_native_tidy.sh" ./ci/test_run_all.sh
> ...
> fatal: destination path '/home/drgrid/bitcoin/ci/scratch/iwyu/include-what-you-use' already exists and is not an empty directory.
ls home/drgrid/bitcoin/ci/scratch/iwyu/include-what-you-use
> ls: cannot access 'home/drgrid/bitcoin/ci/scratch/iwyu/include-what-you-use': No such file or directory
```

Command runs fine on master, so likely something tha
...

💬 ismaelsadeeq commented on issue "25.0 RC Testing Guide Feedback":
(https://github.com/bitcoin/bitcoin/issues/27736#issuecomment-1564126871)
Thank you @evansmj and @D33r-Gee for all your feedback the guide has been updated.

💬 willcl-ark commented on issue "rpc_getblockfrompeer.py intermittent failure: assert_equal(pruneheight, 248); not(249 == 248)":
(https://github.com/bitcoin/bitcoin/issues/27749#issuecomment-1564135165)
FWIW I have seen this error once before running tests locally, but I was making changes to `reconsiderblock` at the time and assumed it was related to that.

👍 hebasto approved a pull request: "refactor: Replace `std::optional<bilingual_str>` with `util::Result`"
(https://github.com/bitcoin/bitcoin/pull/25977#pullrequestreview-1445878314)
ACK 8aa8f73adce72e1ae855b43413c1f65504423cb7, I have reviewed the code and it looks OK.