octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Savannah Status for patches which have been reviewed


From: Nicholas Jankowski
Subject: Re: Savannah Status for patches which have been reviewed
Date: Wed, 18 Jan 2017 10:35:07 -0500

On Wed, Jan 18, 2017 at 10:24 AM, John W. Eaton <address@hidden> wrote:
> On 01/17/2017 11:41 PM, Rik wrote:
>
>> Twice in the last week I've run in to the situation on the bug tracker
>> where a patch has been submitted, and an Octave regular has reviewed it,
>> and it now needs only a quick glance by a second set of eyes before being
>> committed.  What is the appropriate "Status" field to use for this
>> situation?  Is it "Ready for Test"?  Or should a new status entry "Patch
>> Reviewed" be added?
>
>
> A "Patch Reviewed" status might help, at least for sorting purposes, so you
> might as well add it.
>
> It would also be useful to tag a list of people who might be the best
> qualified to review the patch.  But I don't see a way to do that other than
>
>   * adding them to the notification list, but after the initial notice, I
> think it is easily forgotten and I'm not sure if there is a way to search
> for all the bugs where you appear on the notification list
>
>   * assigning the bug to an individual, but that is only for a single
> person, as far as I can tell, so it doesn't really work for requesting that
> multiple people review a patch

maybe an octave.org email alias maintained separately outside of
savannah that can be added as a single 'individual' for something like
that?  could then search for anything assigned to
'address@hidden' ?? I assume savannah has no way to create a
multi-email-address user or aliasing to keep it all internal?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]