
On 23-12-2013 4:11, Bryce W. Harrington wrote:
On Sun, Dec 22, 2013 at 12:01:22PM -0800, Josh Andler wrote:
On Fri, Dec 20, 2013 at 4:56 PM, Bryce Harrington
- Who decides when a given job is "complete"? Do we need to have a separate reviewer role identified (analogous to GSoC mentor)? Should that person get some form of remuneration too? Should the role be interactive with the job performer, or an anonymous pass/fail?
Is this something that the fundraising coordinator should be tasked with or perhaps be the person who proposed the job be the one to review?
Possibly, although I worry that a good bit of time could pass between the proposal, the fundraiser, and the execution of the job, and both the proposer and fundraiser coordinator could drift away. Also, they may not have the technical skill or time/interest in doing the final review.
When is the work committed to trunk? ("committed to trunk" could be a very clear flag for job completion) Some refactoring projects may be best suited as a set of individual patches, instead of one big patch. That makes it much easier for bug tracking I think.
The reviewer role should not be too large I feel (otherwise we will have trouble finding reviewers). One concern is that the reviewer might feel very pressured about his decision. It's about giving someone money, or not, and might end up with tough decisions. (What if the job is complete and all works well, but the coding style conflicts quite a lot with ours... and then the dev says it will take too much time to fix that and... community screams: gogo, commit already! etc...)
regards, Johan