Contributors mailing list archives

contributors@odoo-community.org

Re: Additional information regarding how-to review module

by
Opener B.V., Stefan Rijnhart
- 29/10/2015 17:38:28
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