🤔 mzumsande reviewed a pull request: "index: Fix coinstats overflow"
(https://github.com/bitcoin/bitcoin/pull/30469#pullrequestreview-2998999955)
nit: cummulative -> cumulative in PR description

Looks good to me besides the nits.

This will also need a release note, warning users that existing indexes will be resynced, which will probably take a few days for mainnet.

💬 mzumsande commented on pull request "index: Fix coinstats overflow":
(https://github.com/bitcoin/bitcoin/pull/30469#discussion_r2193373429)
it's commented out, but could also update line 131 above.
Also, `doc/files.md` should be updated.

🤔 furszy reviewed a pull request: "rpc, test: Fix JSON parsing errors in unloadwallet and getdescriptoractivity RPCs"
(https://github.com/bitcoin/bitcoin/pull/32845#pullrequestreview-2999282057)
utACK 529d2ce24570aae426ce19e87cb22f124dd09470

📝 marbcGH opened a pull request: "feature:automatically include the elapsed time in logging"
(https://github.com/bitcoin/bitcoin/pull/32925)
Simple, minimal change to improve logging by including the elapsed time in the "completed" log message even when `msg_on_completion` is set to true. Motivation is to contribute, and improve Bitcoin Core user experience & Bitcoin Core developer experience with a simple log modification.

Current log outputs:
```
LoadBlockIndex completed
```
---

After modification log output:
```
LoadBlockIndex: LoadBlockIndex completed (452.87ms)
```

Current state doesn't include how long the oper
...

📝 Mhwd9-9 opened a pull request: "Create main.yml"
(https://github.com/bitcoin/bitcoin/pull/32926)
<!--
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test improvements or new tests that improv
...

💬 Mhwd9-9 commented on pull request "Create main.yml":
(https://github.com/bitcoin/bitcoin/pull/32926#issuecomment-3050994860)
از ccefe54ccd79f690192cb918ad28359b8ed574f6 دوشنبه ۱۷ سپتامبر ۲۰۰۱ ساعت ۰۰:۰۰:۰۰
از: Mhwd9-9 <152965735+Mhwd9-9@users.noreply.github.com>
تاریخ: چهارشنبه، ۹ ژوئیه ۲۰۲۵ ۰۷:۰۳:۲۱ +۰۳۳۰
موضوع: [PATCH] ایجاد main.yml

---
‎.github/workflows/main.yml | ۱ +
۱ فایل تغییر کرد، ۱ درج انجام شد(+)
ایجاد حالت ۱۰۰۶۴۴ .github/workflows/main.yml

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
حالت فایل جدید ۱۰۰۶۴۴
فهرست ۰۰۰۰۰۰۰۰۰۰۰۰۰..۹۷۵fbec۸۲۵۶d۳
--- /dev/null
++b/.gith
...

💬 Mhwd9-9 commented on pull request "Create main.yml":
(https://github.com/bitcoin/bitcoin/pull/32926#issuecomment-3050995207)
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
حالت فایل جدید ۱۰۰۶۴۴
فهرست ۰۰۰۰۰۰۰۰۰۰۰۰۰..۹۷۵fbec۸۲۵۶d۳
--- /dev/null
++b/.github/workflows/main.yml
@@ -0,0 +1 @@
+ی

💬 luke-jr commented on pull request "RPC/txoutproof: Support including (and verifying) proofs of wtxid":
(https://github.com/bitcoin/bitcoin/pull/32844#discussion_r2193944030)
Renamed to `gentx` (the coinbase is the input only)

👋 luke-jr's pull request is ready for review: "RPC/txoutproof: Support including (and verifying) proofs of wtxid"
(https://github.com/bitcoin/bitcoin/pull/32844)

💬 luke-jr commented on pull request "RPC/txoutproof: Support including (and verifying) proofs of wtxid":
(https://github.com/bitcoin/bitcoin/pull/32844#issuecomment-3051020524)
Reworked this to cover @SomberNight 's requirements and address feedback. Test is now passing.

💬 Mhwd9-9 commented on pull request "Create main.yml":
(https://github.com/bitcoin/bitcoin/pull/32926#issuecomment-3051026881)
من

On چهارشنبه ۹ ژوئیه ۲۰۲۵، ۷:۰۴ DrahtBot ***@***.***> wrote:

> *DrahtBot* left a comment (bitcoin/bitcoin#32926)
> <https://github.com/bitcoin/bitcoin/pull/32926#issuecomment-3050994217>
>
> The following sections might be updated with supplementary metadata
> relevant to reviewers and maintainers.
> Code Coverage & Benchmarks
>
> For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32926.
> Reviews
>
> See the guideline
> <https://github.com/bitcoin/bitcoin/blob/maste
...

✅ maflcko closed a pull request: "feature:automatically include the elapsed time in logging"
(https://github.com/bitcoin/bitcoin/pull/32925)

💬 maflcko commented on pull request "rpc, test: Fix JSON parsing errors in unloadwallet and getdescriptoractivity RPCs":
(https://github.com/bitcoin/bitcoin/pull/32845#issuecomment-3051281022)
review ACK 529d2ce24570aae426ce19e87cb22f124dd09470 🌦

<details><summary>Show signature</summary>

Signature:

```
untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
trusted comment: review ACK 529d2ce24570
...

💬 maflcko commented on pull request "wallet, rpc: add v3 transaction creation and wallet support":
(https://github.com/bitcoin/bitcoin/pull/32896#discussion_r2194156945)
there's also `test/functional/feature_taproot.py:TX_MAX_STANDARD_VERSION = 3`

💬 maflcko commented on pull request "wallet, rpc: add v3 transaction creation and wallet support":
(https://github.com/bitcoin/bitcoin/pull/32896#discussion_r2194159096)
could use strprintf for shorter code? Also missing `{}` around body of the `if`?

✅ Sjors closed a pull request: "rpc: use anti-fee-sniping in send, sendall and walletcreatefundedpsbt"
(https://github.com/bitcoin/bitcoin/pull/32892)

💬 Sjors commented on pull request "rpc: use anti-fee-sniping in send, sendall and walletcreatefundedpsbt":
(https://github.com/bitcoin/bitcoin/pull/32892#issuecomment-3051457537)
Ok, so #28944 uses a completely different approach. It already has a lot of review, so I'll add mine there.

💬 Sjors commented on pull request "wallet, rpc: add anti-fee-sniping to `send` and `sendall`":
(https://github.com/bitcoin/bitcoin/pull/28944#issuecomment-3051460115)
Concept ACK. Discovered this after I tried to implement it myself in #32892 :-)

💬 Evoneem commented on pull request "rpc: use anti-fee-sniping in send, sendall and walletcreatefundedpsbt":
(https://github.com/bitcoin/bitcoin/pull/32892#discussion_r2194341665)
Anti free sniping on future merging please

🤔 Evoneem reviewed a pull request: "rpc: use anti-fee-sniping in send, sendall and walletcreatefundedpsbt"
(https://github.com/bitcoin/bitcoin/pull/32892#pullrequestreview-3000475960)
Anti free sniping on merges