Contributors mailing list archives

contributors@odoo-community.org

Re: Additional information regarding how-to review module

by
Tecnativa. S. L., Pedro M. Baeza
- 26/10/2015 12:08:45
+1

2015-10-26 12:38 GMT+01:00 Joël Grand-Guillaume <joel.grandguillaume@camptocamp.com>:
Hi,


Tha'ts a good question ! I've updated the doc again with the following suggestion:
  • Two reviewers must approve a merge proposal in order to be able to merge it
  • 5 calendar days must be given to be able to merge it
  • A MP can be merged in less that 5 calendar days if and only if it is approved by 3 reviewers. If you are in a hurry just send a mail at contributors@odoo-community.org or ask by IRC (FreeNode oca, openobject channel).
  • At least one of the review above must be from a member of the PSC

Is everybody alright with this ?


Regards,

Joël




On Mon, Oct 26, 2015 at 11:53 AM, Andhitia Rama <andhitia.r@gmail.com> wrote:
@Daniel:

Thank your for your suggestion. Will try it ASAP.

One more question, i understand that PR need 3x +1 from reviewer to be merged. So, any github reviewer +1/-1 will take into consideration whether a PR will be merged/not? Or just +1/-1 from PSC? There are not any documentation in this page or this page regarding this matter.

On Mon, Oct 26, 2015 at 5:07 PM, Daniel Reis <dgreis@sapo.pt> wrote:
Lorenzo is correct, any person can submit reviews, you just need to have 
a Github account.

Not required, but IMO submitting a PR and going through the review 
process first is helpful to get you familiarized with the process.

--Daniel

_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe




--
 
--
 
Andhitia Rama
 

_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe




--


camptocamp
INNOVATIVE SOLUTIONS
BY OPEN SOURCE EXPERTS

Joël Grand-Guillaume
Division Manager


_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe