[Insight-developers] adding authors to Gerrit reviewers

Luis Ibanez luis.ibanez at kitware.com
Sun Nov 14 13:12:13 EST 2010


Matt,

This looks great !

Marcus:

Do you see any reason for not adding this scripts
to the directory:

     ITK/Utilities/DevelopmentSetupScripts


or even to make this an automatic hook ?


   Thanks


        Luis



--------------------------------------------------------------
On Fri, Nov 12, 2010 at 8:58 AM, Matthew McCormick (thewtex) <
matt at mmmccormick.com> wrote:

> Hi,
>
> I scraped together a simple script to automate selection and
> submission of Gerrit reviewers.  Here is a link in case anyone else
> may find it useful:
> https://gist.github.com/674093
>
> Candidates are nominated from former authors of the edited files.
> People probably want to know when I am breaking ... I mean, fixing ;-)
> .. their files, and they are probably the most knowledgeable
> individuals of the issues at hand.  It saves having to pick out
> reviewers manually and the need to re-enter reviewers for every commit
> on a long topic branch.
>
> It has rudimentary user interactivity, but more could be done with it.
>  I don't have time to rigorously fix it up, but if someone is
> interested, go for it.
>
> To use it, run it instead of 'git gerrit-push' on the initial push to
> Gerrit.
>
> Matt
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.html
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20101114/3070686f/attachment.htm>


More information about the Insight-developers mailing list