On Sun, Nov 29, 2015 at 10:09:58PM +0000, Zbigniew Jędrzejewski-Szmek wrote:
Package review is public and happens in the review bug. We also
require (in the sense of having a strong custom, maybe even if it's
not written anywhere explicitly), a checklist style review. This allows
If we want it to be a requirement, we should write it down.
other people to know that not only the reviewer acked the package,
but that they did it with enough diligence.
FWIW, I'm not sure that's really true. The fedora-review tool makes it
very easy to do a low-effort review and still produce a pretty
checklist. (This isn't necessarily a bad thing, but it definitely means
that presence of a checklist doesn't indicate particular diligence.)
--
Matthew Miller
<mattdm(a)fedoraproject.org>
Fedora Project Leader