Michael Schwendt (mschwe...@gmail.com) said: > > * LINK: http://fedoraproject.org/PackageReviewStatus/MERGE.html > > (nirik, 19:59:05) > > * AGREED: : encourage provenpackagers to commit to rawhide fixes for > > merge reviews. said pp's should not be the reviewer. > > Pardon? What exactly is the problem here with a pp being the one to find > an issue *and* commit the fix?
Just saying that the person to commit the fixes found in the review should (IMO) not be the reviewer themselves, even if they happen to be a provenpackager. It allows for a having a second set of eyes on the changes in the case where the maintainer may not have responded. Bill -- devel mailing list devel@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/devel
