[Fedora-packaging] Request for confirmation: Which form is required for a review

[Jason L Tibbitts III]

>>>>> "MR" == Matthias Runge <mrunge at matthias-runge.de> writes:

MR> To be clear, imho there's no need to take back a granted review, the
MR> packager of the reviewed package knows his job well.

Again in the good old days, I was also processing most of the SCM
requests, so I'd effectively stop the process until the review was
re-done.  Occasionally this meant that I did my own review.

MR> I think, a review generates work to do for the reviewer.

Well, theoretically it does.  We have nothing to prevent someone from
just setting the flag without actually doing any review work at all.
Hopefully such stuff would be caught at some point, but really unless
someone is actually looking over all of the tickets this doesn't
happen.  At best someone will notice that a package is in pretty poor
shape, look up the review ticket and notice that it wasn't done well.
But who has time for that?  There are 350+ packages in the review
queue.  Still.

 - J<