On Tuesday, 30 July 2013 9:04 PM, Ken Sharp wrote:

>There's also "Pending".

Hi Ken,

Yes, I know about the 'Pending' status.  The patches page describes this status 
in two ways:

1. The patch is not obviously correct at first glance. Making a more convincing 
argument, preferably in the form of a test case, may help.
2. Waiting for feedback from the main developer in that area.

Both are quite different. Perhaps these should be split with different labels 
as well - particularly as the first requires revisions, while the second 
implies that the patch may be accepted as is.

Anyway, to come back to the point, leaving a patch as 'New' is problematic.  
'Pending' is more informative, yet perhaps not descriptive enough to cover 
patches that stay as 'New' over a 24-hour patch cycle.


Reply via email to