<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif">We've been trialling Github Reviews for some time now and it's time to decide whether we stick with it or go back to Reviewboard. </div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">We're going to have a vote. If you have an opinion on the issue please reply to this email with a +1, 0 or -1, optionally followed by any further thoughts. </div><div class="gmail_default" style="font-family:verdana,sans-serif"><ul><li>+1 means you prefer Github Reviews<br></li><li>-1 means you prefer Reviewboard<br></li><li>0 means you don't mind.<br></li></ul></div><div class="gmail_default" style="font-family:verdana,sans-serif">If you don't mind which review system we use there's no need to reply unless you want to voice some opinions.<br></div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">The voting period starts <b>now</b> and ends my<b> EOD next Friday (October 21)</b>.</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">As a refresher, here are the concerns raised for each option.</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif"><b>Github Reviews</b></div><div class="gmail_default"><ul><li><font face="verdana, sans-serif">Comments disrupt the flow of the code and can't be minimised, hindering readability.</font></li><li><font face="verdana, sans-serif">Comments can't be marked as done making it hard to see what's still to be taken care of.</font></li><li><font face="verdana, sans-serif">There's no way to distinguish between a problem and a comment.</font></li><li><font face="verdana, sans-serif">There's no summary of issues raised. You need to scroll through the often busy discussion page.</font></li><li><font face="verdana, sans-serif">There's no indication of which PRs have been reviewed from the pull request index page nor is it possible to see which PRs have been approved or otherwise.</font></li><li><font face="verdana, sans-serif">It's hard to see when a review has been updated.</font></li></ul><div><font face="verdana, sans-serif"><b>Reviewboard</b></font></div><div><ul><li><font face="verdana, sans-serif">Another piece of infrastructure for us to maintain</font></li><li><font face="verdana, sans-serif">Higher barrier to entry for newcomers and outside contributors</font></li><li><span style="font-family:verdana,sans-serif">Occasionally misses Github pull requests (likely a problem with our integration so is fixable)</span><br></li><li><font face="verdana, sans-serif">Poor handling of deleted and renamed files</font></li><li><font face="verdana, sans-serif">Falls over with very large diffs</font></li><li><font face="verdana, sans-serif">1990's looks :)</font></li><li><font face="verdana, sans-serif">May make future integration of tools which work with Github into our process more difficult (e.g. static analysis or automated review tools)</font></li></ul><div><font face="verdana, sans-serif">There has been talk of evaluating other review tools such as Gerrit and that may still happen. For now, let's decide between the two options we have recent experience with.</font></div><div><br></div><div><font face="verdana, sans-serif">- Menno</font></div></div></div></div>