On Fri, Jul 30, 2010 at 1:10 PM, Darin Adler <[email protected]> wrote: > It’s come to my attention that some webkit-patch or some other script > requires that patches without review contain the string “Unreviewed” > somewhere in their change log message. I personally think this is not > helpful. In my opinion patches that are don’t need review, such as some > simple build fixes, need not list a reviewer, but also need not call > attention to that fact.
The commit-queue requires something like that to make sure it succeeds in filling in the reviewer information from bugs.webkit.org. Running webkit-patch on the command line shouldn't require that. We can figure out another mechanism to make sure the commit-queue fills in the proper reviewer. Adam _______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

