Contributors mailing list archives
Re: Self-merging ruleby
Don't get me wrong, the supervision/control is needed, but keep in mind: every "privilege" has to be proportional with "duty" (not just responsibility, that one is a must for every single party involved in a process), and I'm not sure if in a community of mostly "pro bono" collaborators we can afford "duty". Duty means: if someone has the privilege to do something, he has also the duty to do that thing in a certain time limit in a certain manner. Privilege without duty leads to where most of public administrations are today - paper pushers (or button pushers in our case), and those are the first one getting fired once we start streamlining a business process, since they're not just useless, but they can be considered anchors slowing or even stopping the flow.
+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
Post to: mailto:email@example.com