merge reviews
Kevin Fenzi
kevin at scrye.com
Fri Jul 9 16:07:54 UTC 2010
On Fri, 9 Jul 2010 10:19:14 +0200
Michael Schwendt <mschwendt at gmail.com> wrote:
> On Thu, 8 Jul 2010 14:28:13 -0600, Kevin wrote:
>
> > So, here we are today with 242 still open merge reviews:
> > http://fedoraproject.org/PackageReviewStatus/MERGE.html
> > (Plus a few that were closed when they shouldn't have been).
>
> Dumb question first: Where could I have found the URL of that page?
>
> Several reviewers I know use the cached tracker page, which explicitly
> does NOT include the old merge reviews:
> http://fedoraproject.org/PackageReviewStatus/NEW.html
>
> And that one is linked directly on the Package Review Process page:
> https://fedoraproject.org/wiki/Package_Review_Process#Tracking_of_Package_Requests
Yeah, perhaps that should link to
http://fedoraproject.org/PackageReviewStatus
?
Or both that and NEW?
> For each of the packages, assign its package owner(s) to the review
> ticket, let them perform the review themselves according to Fedora's
> Review Guidelines and when done, set the fedora-review flag to '?'
> and move the ticket to a final tracker. In other words, let the
> owners of these packages indicate that they have (re-)reviewed their
> own package.
Interesting idea. I fear we might get some maintainers who just want
the merge review to go away to just say "reviewed it, everything is
great" -> fedora-review +
without really checking anything.
> Fedora package maintainers must be aware of the packaging guidelines
> anyway when they touch their package spec files, and they also need
> to repeat several checks whenever they includes upgrades (e.g.
> checking for license changes or added code/libs with legal problems).
Indeed. Might be worthwhile to try it out...
kevin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
Url : http://lists.fedoraproject.org/pipermail/devel/attachments/20100709/20c21ae3/attachment.bin
More information about the devel
mailing list