HomePhabricator

Merge #13928: qa: blocktools enforce named args for amount

Description

Merge #13928: qa: blocktools enforce named args for amount

Summary:
cf9ed307e6 qa: blocktools enforce named args for amount (MarcoFalke)

Pull request description:

Since  #13669 changed some signatures, I think it might be worthwhile to enforce named args for primitive types such as amounts.

Backport of Core PR13928

Note that our feature_csv_activation.py differs from Core, so I applied the patch in best spirit with the original PR.

Test Plan:

ninja check-functional

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D8034

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Oct 22 2020, 21:56
jasonbcoxCommitted on Oct 22 2020, 21:56
abc-botPushed on Oct 22 2020, 21:59
Reviewer
Restricted Project
Differential Revision
D8034: Merge #13928: qa: blocktools enforce named args for amount
Parents
rABC47e2944da10c: [build-bot] Fix database test when an extension is not present on the database…
Branches
Unknown
Tags
Unknown