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 <mat...@fedoraproject.org> Fedora Project Leader -- devel mailing list devel@lists.fedoraproject.org http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org
