🚀 hebasto merged a pull request: "Fix segfault when shutdown during wallet open"
(https://github.com/bitcoin-core/gui/pull/693)

💬 pinheadmz commented on issue "Where are the Developers keys to Install Bitcoin Core ?":
(https://github.com/bitcoin/bitcoin/issues/27346#issuecomment-1485266336)
The keys are here: https://github.com/bitcoin/bitcoin/blob/master/contrib/verify-commits/trusted-keys

I'll see if we need to update the docs on the website

💬 hernanmarino commented on pull request "Debug Console implementation of generate method":
(https://github.com/bitcoin-core/gui/pull/692#issuecomment-1485269836)
> Could this PR be separated in two commits:
>
> * refactoring and introducing the `RPCExecutor::executeConsoleOnlyCommand()` function
> * actually adding support for the `generate` command`
>
> ?

Yes, will do ASAP.

💬 pinheadmz commented on issue "Where are the Developers keys to Install Bitcoin Core ?":
(https://github.com/bitcoin/bitcoin/issues/27346#issuecomment-1485272621)
correction: they were moved to https://github.com/bitcoin-core/guix.sigs/tree/main/builder-keys and the original locaiton was deleted in https://github.com/bitcoin/bitcoin/pull/26598

💬 pinheadmz commented on issue "Where are the Developers keys to Install Bitcoin Core ?":
(https://github.com/bitcoin/bitcoin/issues/27346#issuecomment-1485280130)
Closing this as duplicate of https://github.com/bitcoin-core/bitcoincore.org/issues/945. Discussion should move to the bitcoincore.org repo

✅ pinheadmz closed an issue: "Where are the Developers keys to Install Bitcoin Core ?"
(https://github.com/bitcoin/bitcoin/issues/27346)

💬 hernanmarino commented on pull request "build, qt: Fix handling of `CXX=clang++` when building `qt` package":
(https://github.com/bitcoin/bitcoin/pull/27314#issuecomment-1485282916)
I can confirm with clang-15.
```
Ubuntu clang version 15.0.7
Target: x86_64-pc-linux-gnu
Thread model: posix
```

💬 hebasto commented on pull request "depends: fontconfig 2.14.2":
(https://github.com/bitcoin/bitcoin/pull/27301#discussion_r1149392363)
Is `-Wno-int-conversion` really required to compile with clang-16?

💬 hebasto commented on pull request "depends: fontconfig 2.14.2":
(https://github.com/bitcoin/bitcoin/pull/27301#issuecomment-1485289625)
> depends: fontconfig 2.14.2

Update the title?

💬 1440000bytes commented on pull request "rpc, wallet: add ability to retrieve all address book entries":
(https://github.com/bitcoin/bitcoin/pull/26174#issuecomment-1485309914)
Concept ACK

💬 hebasto commented on pull request "Warn when sending to already-used Bitcoin addresses":
(https://github.com/bitcoin-core/gui/pull/562#issuecomment-1485310320)
Closing this due to lack of activity. Feel free to reopen.

✅ hebasto closed a pull request: "Warn when sending to already-used Bitcoin addresses"
(https://github.com/bitcoin-core/gui/pull/562)

💬 hebasto commented on pull request "Make error message layout consistent":
(https://github.com/bitcoin-core/gui/pull/560#issuecomment-1485313776)
Closing this due to lack of activity. Feel free to reopen.

✅ hebasto closed a pull request: "Make error message layout consistent"
(https://github.com/bitcoin-core/gui/pull/560)

💬 hebasto commented on pull request "Network graph improvements":
(https://github.com/bitcoin-core/gui/pull/559#issuecomment-1485316876)
Closing this due to lack of activity. Feel free to reopen.

💬 hebasto commented on pull request "Network graph improvements":
(https://github.com/bitcoin-core/gui/pull/559#issuecomment-1485316876)
Closing this due to lack of activity. Feel free to reopen.

✅ hebasto closed a pull request: "Network graph improvements"
(https://github.com/bitcoin-core/gui/pull/559)

💬 hebasto commented on pull request "network graph - show/hide panels based on window width/height":
(https://github.com/bitcoin-core/gui/pull/540#issuecomment-1485322497)
Closing this due to lack of activity. Feel free to reopen.

✅ hebasto closed a pull request: "network graph - show/hide panels based on window width/height"
(https://github.com/bitcoin-core/gui/pull/540)

💬 1440000bytes commented on pull request "init: Error if ignored bitcoin.conf file is found":
(https://github.com/bitcoin/bitcoin/pull/27302#issuecomment-1485326965)
> Show an error on startup if a bitcoin datadir that is being used contains a bitcoin.conf file that is ignored

1. Why is it ignored?
2. Any downsides for using bitcoin.conf from datadir if conf argument isn't used?

💬 furszy commented on pull request "wallet: finish addressbook encapsulation":
(https://github.com/bitcoin/bitcoin/pull/26836#discussion_r1149415708)
I prefer the current code, it's more readable.
The curly braces initializer is primarily useful to prevent narrowing (which is the implicit conversion of arithmetic values). Here, there is no loss of accuracy, we are simply initializing a boolean from a logical statement.