pull down to refresh

I see you are a maintainer now. Nice! How long did it take?

(I assume you are a maintainer because you approved my PR)

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.

https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md

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:

https://postimg.cc/nsPPRzp0

reply

Ahh, no, that's just what GitHub writes if you select "Approve" instead of "Comment" at "Files changed > Review changes".

https://imgur.com/gGktFJ3.png

reply

Ah, I see haha

reply