Thanks Arvind, I prefer RTC with timeout. If we decide on timeout, what is the suitable timeout period and how can we manage it for different patches (some patches may require more time than others for review)? Is the timeout measured from review submission or from last activity on the review? Any ideas?
Best Regards Ahmed On Fri, Aug 12, 2011 at 10:59 AM, Arvind Prabhakar <[email protected]>wrote: > Hi All, > > We are starting to see some traction in JIRA and patch activity. I > believe now is a good time for us to put a formal policy in place that > guides the overall review and commit process. Different projects have > adopted different ways of addressing this, but at a high level there > are two - Review-Then-Commit (RTC) style, and Commit-Then-Review > (CTR). > > Lets discuss this to bring out various point of views and then do a > formal vote on the candidate policy that is acceptable to the > majority. > > My thoughts: I prefer RTC with timeout provisions. Specifically, I > feel that every change must get reviewed and if the reviewers do not > respond within a certain time, the change can be committed. > > Please share your thoughts, comments and concerns on this. > > Thanks, > Arvind >
