Note that Electrum ABC switched back to using the base unit (now XEC) in the payment URIs. And the node software does the same.

I also checked that this typo isn't in Bitcoin Core, it originated with ABC here: https://reviews.bitcoinabc.org/D3474

Thanks. I fixed the description.

The PR number is wrong

Fix the circular dependency

Use a script instead

It was wrong in multiple ways :)

Investigating how to tweak the parameters to avoid undoing existing multiline imports when they would be shorter than 80 characters on a single line.

it needs splitting, and I'm still investigating whether it is possible to preserve the multiline imports that we already have, even if they would be shorter than 80 characters on a single line.

Verify [Add|Sub]Int63OverflowEmulated and [Add|Sub]Int63Overflow overflow for the same operands and produce the same result for for non-overflowing operands.

Actually fix the typo in "unreachable" in OP_CHECKMULTISIG

Can you also remove the extra e after the h, while you are at it.

Ok, nevermind, the assumptions are there, but we have a file for that. This avoids baking in contradictory assumptions.

Can you please time before/after this linter ? External linters tend to be costly.

This is way too big to be reviewed, please split

No need to wait for anything before landing

I will wait for D10124 to land before rebasing and landing this.

Add comment for the int64_t == long long int assertion.

fix import ordering