Re: Process Change: Package Reviews with Flags

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

 



On Wed, 31 Jan 2007 09:55:25 -0500, Warren Togami wrote:
> 2) ASSIGNED begins to nobody@xxxxxxxxxxxxxxxxxx   ASSIGNED to reviewer 
> during the review, ASSIGNED to package owner after review.  ASSIGNED is 
> to whoever needs to be doing work in the next step of getting the 
> package out.

Makes some amount of sense, thinking about it...

> Perhaps #2 makes more sense, but is it too confusing?

Well, I got used to the current schema, so this new schema is confusing
me a bit anyway...

> Either way, Flags track *who* did the review and *who* approved it, and 
> flags are independently query-able.

Yes.  But they won't show who the package owner is AFAICT...  So I
think the owner should be the assignee at least when the package is
approved.  Dunno how we will know who the package owner is during
review with option 2.  I think you should name the owner in the
initial comment in this case.

Did you intend to still have all accepted packages block
FE-ACCEPT ?

					C

--
Fedora-maintainers mailing list
Fedora-maintainers@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers

--
Fedora-maintainers-readonly mailing list
Fedora-maintainers-readonly@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers-readonly

[Index of Archives]     [Fedora Users]     [Fedora Development]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux