[Insight-developers] Gerrit Reviewers : Still Looking for more.

[Luis Ibanez]

Cory,

Thanks for updating the Wiki,

However,
It looks like you undid the table that Dan created.

The Wiki is not very good at managing conflicts
(e.g. it doesn't do merges)...

Here is the history
http://www.itk.org/Wiki/index.php?title=ITK%2FGerrit%2FReviewers&action=historysubmit&diff=32420&oldid=32409

So if you two started editing the page at the same
time, and Dan saved his changes (with the table),
then you saved your changes (that started from
the version without the table), then your changes
removed Dans version.

So, let me restore a union of both versions,
and we can reorganize it from there.

I see your point that it makes more sense to
sort reviewers by their areas of expertise and
interest than by their names.

I'll see how I can reformat it.


     Thanks


          Luis



------------------------------------------------------------------
On Thu, Oct 14, 2010 at 1:16 AM, Cory Quammen <cquammen at cs.unc.edu> wrote:

> Dan and Luis,
>
> I thought it would be nice to reorganize the wiki page to list the
> areas of expertise as headings and the developers with a particular
> expertise under the appropriate heading. That way, if I submitted a
> change related to image registration to gerrit, I could click on the
> link for image registration in the table of contents at the top of the
> wiki page and jump down to the list of qualified reviewers.
>
> Feel free to revert this change if you prefer the other organization.
>
> Cory
>
> On Wed, Oct 13, 2010 at 11:08 PM, Dan Mueller <dan.muel at gmail.com> wrote:
> > Hi Luis,
> >
> > I found the wiki page a little hard to read, so I refactored using a
> > table. Hope that is ok...
> >
> > Cheers, Dan
> >
> > On 13 October 2010 23:27, Luis Ibanez <luis.ibanez at kitware.com> wrote:
> >>
> >> Thanks to all of you who volunteered to help
> >> with the review process.
> >>
> >> The current list of reviewers has been posted
> >> in the Wiki at:
> >>
> >> http://www.itk.org/Wiki/ITK/Gerrit/Reviewers
> >>
> >> The section of reviewer's "expertise / interest"
> >> is currently empty. It is probably better if every
> >> reviewer put some entries under her/his name:
> >>
> >> http://www.itk.org/Wiki/ITK/Gerrit/Reviewers#Expertise_.2F_Interests
> >>
> >>
> >> --
> >>
> >> We are still in need of 5 to 10 more reviewers.
> >>
> >> This is critical to ensure that the incoming code
> >> gets to be checked by many eyes, and that we
> >> all engage in the process of eliminating potential
> >> incoming bugs.
> >>
> >>
> >> This is direct application of Rule #2 of Open Source:
> >>
> >>   "Given enough eyeballs, all bugs are shallow"
> >>
> >>
> >>
> >>
> >> Please let us know if you want to participate,
> >>
> >>
> >>       Thanks
> >>
> >>
> >>             Luis
> > _______________________________________________
> > 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
> >
>
>
>
> --
> Cory Quammen
> Computer Integrated Systems for Microscopy and Manipulation (CISMM)
> Department of Computer Science
> University of North Carolina at Chapel Hill
> http://www.cs.unc.edu/~cquammen <http://www.cs.unc.edu/%7Ecquammen>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20101014/82417121/attachment.htm>