Re: RFC: Review with Flags (Version 3)

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

 



Christian Iseli wrote:
On Mon, 05 Feb 2007 18:41:23 -0500, Warren Togami wrote:
2) BLANK means review not requested. ? means review requested. This makes it easier to query, because querying for BLANK is a bit inefficient.
(Good idea, bad idea?)

bad idea IMHO.  Filing the ticket *does* mean I want a review.  I much
prefer nirik's suggestion:
blank -> I want a reviewer
? -> normal review under way (can use NEEDINFO if wanted)
+ -> ACCEPT
- -> something bad happened (stalled review, bad license, ...)

The rest sounds fine.


+1 Although I'm not sure about the BLANK vs ? difference, don't we already have NEW versus ASSIGNED for this?

Regards,

Hans

--
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