doc: Use clang-tidy named args

[maflcko]

Mirror of bitcoin/bitcoin#24786

[sipa]

ACK 20854df

[maflcko]

Is the goal to mirror all changes or is the goal to remove the Bitcoin Core stuff here, now that the stuff has been added to Bitcoin Core?

[sipa]

I think the goal is to first get everything here into Bitcoin Core (except the compiler/website). Selective deleting things here I think would complicate testing (and remove the ability to e.g. compile the miniscript compiler here). @darosior thoughts?

[darosior]

I agree i don't think deleting the code from here makes a lot of sense. I expect a bit more movements here with Tapscript, but after that i don't expect the code shared between this repo and the Bitcoin Core one to change much.
So eventually i think it'll be worth the small maintenance burden of keeping this repo in sync with Bitcoin Core. And who knows, someone might want to make it an independent library at some point? :-)