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
d.l.neto@onestein.nl
- 26/08/2015 16:59:24
+1 For not allow self-merging

I'm agree with every word of Joël.

Regards,
Diego


On 08/26/2015 10:37 AM, Joël Grand-Guillaume wrote:
<blockquote cite="mid:CACk3wO+6EPcEJopLKoWyJL4QK-xZXSA18qiBrfB0tW8530a4rw@mail.gmail.com" type="cite">
+1 to not allow self-merging.

This should be the rule everybody respect. Moreover that's a common rule in all open source ecosystem/project and for good reasons.

Now, If a core maintainer do it for good reasons (typo fix, little nits, etc..) that's alright, but that must remain the exception, not ther ule.

Regards,

Joël


On Tue, Aug 25, 2015 at 9:08 AM, Daniel Reis <dgreis@sapo.pt> wrote:
+1 for not self-merging

I've also been asked to not do it, I've avoided it since then, and think 
it works well.
Merging is a last checkpoint, and should be done by an independent 
manitainer.
There are many people with write access, adding a comment asking for it 
to be merged is not an issue.

--DR

Às 13:22 de 23/08/2015, Pedro Manuel Baeza Romero escreveu:
> 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

_______________________________________________
Mailing-List: http://odoo-community.org/groups/oca-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
Business Solutions

+41 21 619 10 28


-- 
Diego Luis Neto
SW development team manager

P: +31 88 627 0425
M: +31 6 3808 6589
S: diegoluis.neto
E: d.l.neto@onestein.nl

Onestein B.V.
P: +31 (0) 88 6270400
W: www.onestein.eu
T: onestein_erp
In: onestein

Amsterdam - Keizersgracht 62-64
Oosterhout - Beneluxweg 37
Antwerpen - Brusselsestraat 51

Reference