incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Nalley <da...@gnsa.us>
Subject Re: Patches review
Date Mon, 25 Jun 2012 22:58:07 GMT
Hi folks,

It's been about a week since Review Board was stood up.

Anyone have any comments?

I have a couple - first compared to gerrit, ReviewBoard def lacks a
few features, though it's virtually zero effort for us to maintain. I
think it does provide a nice centralized place to look for patches,
but at the moment, that's about it. So it's better than email (where
50% of the patches submitted ended up being mangled).

I personally would like to see something pretty heavily automated to
at least verify:
* Patch applies cleanly
* Patched code builds

I'd also like for it to automate some testing at some point - though
the above two seem like lower hanging fruit.
So that bears the question do we want to try and stand up gerrit?
Anyone in love with ReviewBoard?

--David

Mime
View raw message