<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 6, 2014 at 8:01 AM, Donald Stufft <span dir="ltr"><<a href="mailto:donald@stufft.io" target="_blank">donald@stufft.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="h5"><br><div><br></div></div></div><div>One potential solution is Phabricator (<a href="http://phabricator.org" target="_blank">http://phabricator.org</a>) which is a gerrit like tool except it also works with Mercurial. It is a fully open source platform though it works on a “patch” bases rather than a pull request basis. </div></div></blockquote><div><br></div><div>I've been pleasantly unsurprised with the ReviewBoard CLI tools (RBtools):</div><div><br></div><div>* <a href="https://www.reviewboard.org/docs/rbtools/dev/">https://www.reviewboard.org/docs/rbtools/dev/</a><br></div><div>* <a href="https://www.reviewboard.org/docs/codebase/dev/contributing-patches/">https://www.reviewboard.org/docs/codebase/dev/contributing-patches/</a><br></div><div>* <a href="https://www.reviewboard.org/docs/manual/2.0/users/">https://www.reviewboard.org/docs/manual/2.0/users/</a><br></div><div><br></div><div>ReviewBoard supports Markdown, {Git, Mercurial, Subversion, ... }, full-text search</div><div><br></div><div>* <a href="https://wiki.jenkins-ci.org/display/JENKINS/Reviewboard+Plugin">https://wiki.jenkins-ci.org/display/JENKINS/Reviewboard+Plugin</a></div><div>* [ <a href="https://wiki.jenkins-ci.org/display/JENKINS/Selenium+Plugin">https://wiki.jenkins-ci.org/display/JENKINS/Selenium+Plugin</a> ]</div><div>* <a href="https://github.com/saltstack/salt-testing/blob/develop/salttesting/jenkins.py">https://github.com/saltstack/salt-testing/blob/develop/salttesting/jenkins.py</a></div><div>  * GetPullRequestAction</div><div>  * <a href="https://wiki.jenkins-ci.org/display/JENKINS/saltstack-plugin">https://wiki.jenkins-ci.org/display/JENKINS/saltstack-plugin</a> (spin up an instance)</div><div>  * <a href="https://github.com/saltstack-formulas/jenkins-formula">https://github.com/saltstack-formulas/jenkins-formula</a></div><div>  * <a href="https://github.com/saltstack/salt-jenkins">https://github.com/saltstack/salt-jenkins</a></div><div> </div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Terry spoke about CLAs, which is an interesting thing too, because phabricator itself has some workflow around this I believe, at least one of the examples in their tour is setting up some sort of notification about requiring a CLA. It even has a built in thing for signing legal documents (although I’m not sure if that’s acceptable to the PSF, we’d need to ask VanL I suspect). Another neat feature, although I’m not sure we’re actually setup to take advantage of it, is that if you run test coverage numbers you can report that directly inline with the review / diff to see what lines of the patch are being exercised by a test or not.</div></div></blockquote><div><br></div><div>AFAIU, these are not (yet) features of ReviewBoard (which is written in Python).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>I’m not sure if it’s actually workable for us but it probably should be explored a little bit to see if it is and if it might be a good solution. They also have a copy of it running which they develop phabricator itself on (<a href="https://secure.phabricator.com/" target="_blank">https://secure.phabricator.com/</a>) though they also accept pull requests on github.</div></div></blockquote><div><br></div><div>What a good looking service.</div></div></div></div>