qa: blocktools enforce named args for amount #13928

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1808-qaEnforceNamedArgs changing 8 files +37 −41

MarcoFalke commented at 5:28 PM on August 9, 2018: member

Since #13669 changed some signatures, I think it might be worthwhile to enforce named args for primitive types such as amounts.
qa: blocktools enforce named args for amount cf9ed307e6
MarcoFalke added the label Tests on Aug 9, 2018
MarcoFalke commented at 5:28 PM on August 9, 2018: member

cc @conscott
conscott commented at 6:35 PM on August 9, 2018: contributor

Tested ACK cf9ed307e6efd2b63d54c74ca8bdd236028fd9dc

Thanks for updating.
fanquake commented at 9:54 AM on August 13, 2018: member

Compiled and ran the functional tests: test/functional/test_runner.py

utACK cf9ed30
ken2812221 referenced this in commit b8eb0dfde4 on Aug 13, 2018
MarcoFalke merged this on Aug 13, 2018
MarcoFalke closed this on Aug 13, 2018
MarcoFalke deleted the branch on Aug 13, 2018
jasonbcox referenced this in commit 7eb5e1e2e4 on Oct 22, 2020
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels