RFC: Review with Flags (Version 5)

[Warren Togami]

Till Maas wrote:
> On Dienstag 20 Februar 2007, Warren Togami wrote:
> 
>> Remove the use of FE-NEW, FE-REVIEW, and FE-ACCEPT.
> 
> For this fedora-review{BLANK,?,+} would be enough and when it is not needed to 
> change the blocking bugs anymore, it does not make it more complicated, than 
> it is now. Is there als an advantage of using "fedora-review-" instead of 
> only using "NEEDINFO" when someone has to do something? And does assigning 
> to "nobody at fedoraproject.org" or the reviewer instead of 
> using "fedora-review{BLANK,?}" and changing to "CLOSED RAWHIDE" instead of 

Eh?

> fedora-review+ not scale, too? I think it is odd to mark the state of a 
> ticket with two ways which both need to be done manually, when only one may 
> be sufficient.
> 

fedora-review state indicates the state of the review.
BLANK (not yet reviewed)
?     (currently under review)
-     (needs work!)
+     (approved)

Package Review APPROVED does not mean the package is done.

Bugzilla CLOSED should be clear.  CLOSED means the package itself is 
done, imported, build and tested.

Yes, this still is confusing and not streamlined, but it is better than 
the earlier process.  We are limited by the tools, and we need something 
NOW.

*PLEASE* help us in the design and implementation of the Package 
Database so that we may make this even simpler in the future.

Warren Togami
wtogami at redhat.com