cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Boris Schrijver <bo...@pcextreme.nl>
Subject PR's and the 2 LGTM Story
Date Mon, 14 Dec 2015 15:58:47 GMT
Hi all,

Just to make this clear once more. A PR needs two LGTM's. One of them needs to
run the integration- and unit-tests. While the other does code review.

This is a the minimal requirement. More is always welcome, anything less will be
reverted.

I just enforced this policy [1]. Any active committer has at all times the right
to do so.

[1]
https://github.com/apache/cloudstack/commit/a00fef8c332ebaede32c46cbf3065f4acaa91f02

-- 

Met vriendelijke groet / Kind regards,

Boris Schrijver

PCextreme B.V.

http://www.pcextreme.nl/contact
Tel direct: +31 (0) 118 700 215

Mime
View raw message