pull down to refresh
31 sats \ 5 replies \ @kristapsk 9 Mar 2023 \ on: My first bitcoin PR bitcoin
This was my first Bitcoin Core PR (was a result of me debugging my scripts that use bitcoin-cli). https://github.com/bitcoin/bitcoin/pull/13032
I see you are a maintainer now. Nice! How long did it take?
(I assume you are a maintainer because you approved my PR)
reply
No, I'm not a Bitcoin Core maintainer. Anybody can ACK or NACK commits in a PR after code review / testing (and the more people do this, the better). Maintainers are the only ones who can merge them at the end. But they are not supposed to use these rights if there are a lot of NACKs or not enough ACKs for more complex code changes.
reply
Yes, I know about the ACKs and NACKs. But I don't see an approve button. So I thought you have some special rights:
reply
Ahh, no, that's just what GitHub writes if you select "Approve" instead of "Comment" at "Files changed > Review changes".
reply
Ah, I see haha
reply