1 |
On Thursday 16 January 2014 12:44:57 Alexander Berntsen wrote: |
2 |
> But basically, if someone authors a non-trivial patch, that person |
3 |
> should *never* push themselves. Whoever reviews it should push it, |
4 |
> adding the Reviewed-by field. The reviewer should also get an ACK by |
5 |
> the team lead (via IRC or whatever) and add that to the commit before |
6 |
> pushing. |
7 |
|
8 |
err, what ? that level of process really doesn't make sense here. the |
9 |
project isn't large enough or have enough active people to require it. |
10 |
|
11 |
any dev with commit access is trusted to do the right thing and to shepherd |
12 |
their work through. |
13 |
-mike |