Contributors mailing list archives

contributors@odoo-community.org

Re: Additional information regarding how-to review module

by
Elico Corp, Eric Caudal
- 29/10/2015 22:35:38
+1
--
Eric Caudal [Founder and CEO]
Skype: elico.corp. Phone: + 86 186 2136 1670 (Cell), + 86 21 6211 8017/27/37 (Office)
Elico Shanghai (Shenzhen/Singapore) Odoo Gold Partner, best Odoo Partner 2014 for APAC
On 10/30/2015 01:53 AM, Stefan Rijnhart wrote:
<blockquote cite="mid:56325994.1000203@opener.am" type="cite">
Yes, I was wondering about that as I don't think I am a PSC myself in
many projects, but I like to be active in the domain that I am currently
working on, moving from project to project. So +1 for Daniel's reasoning.

Stefan.


On 29-10-15 10:08, Daniel Reis wrote:
>
> Joël,
>
> This is an implicit requirement to reviews: only a person with write
> access can merge the PR.
> There is no chance for two "rogue" reviewers to approve a PR and have
> it merged - one person with write access is always needed to merge the PR.
>
> In principle this should be someone from the PSC, and in practice can
> be any OCA admin.
>
> IMO would should keep that flexibility and keep it as it is because:
>
> - Many project have no regular activity from their PSC, and may not
> even have a PSC formed.
> - If having an active PSC, I see no issues on having a non-PSC to de
> the final merge for fixes, minor changes or even complex reviews where
> the PSC already gave the OK.
>
> IMO this adds additional complexity that is not needed and won't bring
> any true benefits.
> I suggest instead to clarify that a person with write access is
> required for a review or final merge.
>
> --Daniel
>

-- 
Opener B.V. - Business solutions driven by open source collaboration

Stefan Rijnhart - Consultant/developer

mail: stefan@opener.am
tel: +31 (0) 20 3090 139
web: https://opener.am

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