[Owncloud] [QA-Announcement] Code Reviews to increase quality

Sam Tuke samtuke at owncloud.com
Mon Oct 29 23:59:21 UTC 2012


On 10/29/2012 11:43 PM, Jan-Christoph Borchardt wrote:
> TL;DR: Do not directly push to core master, and instead use pull
> requests.

How much of the procedure is enforced, and how much relies upon convention?
Who counts as a "reviewer" - should only certain people fulfil this role
(are there specific reviewers for specific files / apps)?
Can people who aren't members of ownCloud on GitHub also act as reviewers?
Do +1s have to take a specific form, or are comments like "looks good"
sufficient?
Is it now literally not possible to commit directly to core master?

Great progress guys, awesome work!

Sam.



More information about the Owncloud mailing list