[webkit-dev] Review states

Maciej Stachowiak mjs at apple.com
Wed Jun 17 12:54:04 PDT 2009


On Jun 17, 2009, at 12:52 PM, Maciej Stachowiak wrote:

>
> On Jun 17, 2009, at 11:45 AM, Ojan Vafai wrote:
>
>> This is surely bike-shedding, but...
>>
>> On Wed, Jun 17, 2009 at 8:14 AM, Maciej Stachowiak <mjs at apple.com>  
>> wrote:
>> Review
>> Rejected (for the rare case where the concept of a patch tries to  
>> do is completely wrong; often in this case the bug will also be  
>> INVALID)
>> Revise and Resubmit
>>
>> How about "Needs work"? I find "resubmit" a confusing word as  
>> commit and submit are often used interchangeably.
>
> Just "Revise" would be ok with me, but I wanted to draw more of a  
> distinction from "Commit with Changes". "Revise and Repost" maybe,  
> since we usually speak of "posting" a patch, rather than  
> "submitting" it? Or just "Post a New Patch".
>
>>
>> Commit with Changes (probably this should only be used when the  
>> patch submitter is a commiter)
>> Commit
>>
>
>
> Anyway, to make any changes like this, we'll have to work around  
> some of the technical limitations of bugzilla. A number of us have  
> experience hacking bugzilla, so this is totally feasible. If there  
> were an open source bug tracker that met our needs much better, to  
> the point that transition costs would be lower than hacking  
> bugzilla, then we could consider it. But that depends on having a  
> concrete alternative that is actually better.

I forgot to mention - there should probably be a final "Checked In"  
state. I think that would help smooth out the workflow when people  
attach multiple patches to a single bug. It seems more clear than  
unflagging.

Cheers,
Maciej

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090617/969c5ed2/attachment.html>


More information about the webkit-dev mailing list