1. Mailing Lists
  2. Contributors
  3. Re: Self-merging rule

Contributors mailing list archives

contributors@odoo-community.org

Browse archives

Re: Self-merging rule

Re: Self-merging rule

Avatar

Re: Self-merging rule

by
Open Source Integrators, Maxime Chambreuil
- 23/08/2015 21:05:34
+1

Anybody can review and put a +1. Having 3 +1 from less experienced contributors does not mean you can merge your own PRs.
--
Maxime Chambreuil
+1 (514) 276-5468 #126

----- Le 23 Aoû 15, à 14:53, Graeme Gellatly <gdgellatly@gmail.com> a écrit :

Hi

I'm with Stefan on this. People with merge rights know everybody else with merge rights and are respected.  It should be no big deal to just ask one on Irc to take a quick check and merge. 

The only exceptions should be if one of the approvals was by someone with merge rights or it is non Odoo code,  eg new tests,  Travis config but even then I'd prefer not. 

The act of merging is more than just simply pressing a button it represents an absolute approval by one of the repositories most senior members.  Self merging undermines that.

Well at least to me. 


Sent from TypeMail

On 24 Aug 2015, at 12:22 AM, Pedro Manuel Baeza Romero <pedro.baeza@gmail.com> wrote:
Hi all,

I faced some days ago a complaint from Stefan Rijnhart about myself merging one of my PRS when the number of approvals was the appropiate. This is an old OCA rule that was applied on old Launchpad times.

This rule is not even in the current OCA contributing guidelines (https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md#review), but he insists on bringing this topic to the mailing list.

I see no special reason to apply again this rule nowadays, because thanks to the merge button, the task doesn't involve any special risk to screw up things. In Launchpad, there was this risk, as the merge implies some manual operations that can lead to a bad merging, but as I said, today this is not a problem.

What the others think about this rule?

Regards.

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

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


Reference