[development] RTBC - how does it work?
gerhard at killesreiter.de
Mon Jul 2 07:30:07 UTC 2007
-----BEGIN PGP SIGNED MESSAGE-----
Khalid Baheyeldin schrieb:
> On 7/1/07, Doug Green <douggreen at douggreenconsulting.com> wrote:
>> I searched the docs and couldn't find the protocol for this. How does a
>> patch get to be RTBC? Is this a peer review thing or is it limited to a
>> handful of people. The only thread I found was this one, which is
>> a work-in-progress, and one that we've moved past:
> Peer review by those who test it and find it works.
"It works" isn't all that's needed. It should follow the coding style as
well as being done "the Drupal way"(tm).
> Definitely not limited to a chosen few.
Well, of course anybody can set the status to rtbc. But since a core
committer will look at who set it to that status, it will help if
somebody does it who is known to not lightheartedly set everything to
rtbc just because he likes the feature. It also helps if the person who
sets the status has a known experience in the area the patch addresses.
So, indeed in some sense it is limited to a few, though these aren't
really chosen in some way.
The status of a "trustworthy patch reviewer" can be earned. It is a lot
of hard work.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
-----END PGP SIGNATURE-----
More information about the development