Robert Haas <robertmhaas@gmail.com> wrote:
> While I was at it, I implemented a feature I've been wanting for a
> while: I made the "Status Summary" line at the top of the
> CommitFest page have links to filter by status.
Very nice. I was going to ask to have "Ready for Committer" split
out to its own section, but with this filtering, it's probably not
worth the bother. This change will be very nice for CF managers.
While we're on the topic of CF app enhancements, I often wished that
the date of the last change to the "Reviewers" column would show
underneath the name(s) where the value was not empty and the date
was later than both the "Last Activity" date and the start of the
CF. (Either that or count a non-NULL value set into this column as
a reason to set the current date into "Last Activity", but I like
the extra date better.)
It occasionally seems as though WiP patches are different enough
that there should be a more systematic was to flag them and count
them, but I can't think of any concrete way to do that which doesn't
introduce more problems than it would fix.
And I still think that a link back to the CommitFest Wiki page might
prevent the occasional gaff by people new to the application, but
that assumes they'd follow the link and read up on the process
before jumping in with entries in the app. The two most common
issues seem to be putting a URL in the Message-ID field, and putting
a whole review into the comment text rather than a brief summary and
a link to a post with the review. Occasionally people failed to set
a new status when they should have done after linking in a new patch
or review.
-Kevin