<div dir="ltr"><div>This lists the ReviewBoard workflow steps for a pre-commit workflow:</div><div><br></div><div><a href="https://www.reviewboard.org/docs/manual/dev/users/getting-started/workflow/">https://www.reviewboard.org/docs/manual/dev/users/getting-started/workflow/</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 6, 2014 at 7:55 PM, Wes Turner <span dir="ltr"><<a href="mailto:wes.turner@gmail.com" target="_blank">wes.turner@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Sat, Dec 6, 2014 at 7:23 PM, Wes Turner <span dir="ltr"><<a href="mailto:wes.turner@gmail.com" target="_blank">wes.turner@gmail.com</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 dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>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><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></span><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/" target="_blank">https://www.reviewboard.org/docs/rbtools/dev/</a><br></div><div>* <a href="https://www.reviewboard.org/docs/codebase/dev/contributing-patches/" target="_blank">https://www.reviewboard.org/docs/codebase/dev/contributing-patches/</a><br></div><div>* <a href="https://www.reviewboard.org/docs/manual/2.0/users/" target="_blank">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></div></div></blockquote><div><br></div></span><div><a href="https://www.reviewboard.org/docs/manual/dev/extending/" target="_blank">https://www.reviewboard.org/docs/manual/dev/extending/</a><br></div><div><br></div><div>* "<a href="https://www.reviewboard.org/docs/manual/dev/extending/extensions/" style="font-size:13px;color:rgb(0,0,238);font-weight:bold;text-decoration:none;font-family:Verdana,Arial,Helvetica,sans-serif;background-color:rgb(246,246,246)" target="_blank">Writing Review Board Extensions</a>"</div><div>* "<a href="https://www.reviewboard.org/docs/manual/dev/extending/auth-backends/" style="font-size:13px;color:rgb(0,0,238);font-weight:bold;text-decoration:none;font-family:Verdana,Arial,Helvetica,sans-serif;background-color:rgb(246,246,246)" target="_blank">Writing Authentication Backends</a>"</div><span class=""><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 dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><span><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></span><div>AFAIU, these are not (yet) features of ReviewBoard (which is written in Python).</div></div></div></div>
</blockquote></span></div><br></div></div>
</blockquote></div><br></div>