Le 22/02/2011 12:46, Chris Nighswonger a écrit :
Chris can correct me if I'm wrong....
Once the patch has been pushed to a QA branch it is "Pushed for QA."
If it fails QA it should be "Failed QA."
If it passes it should be "Needs sign off" or "Signed off."
Once pushed to master it is "Patch Pushed."
If at any time it fails a test the status should return to "Failed QA." That's how I read it too, but when "patch pushed", there's something in master and something that is wrong. So it should not just be "failed QA" because in this status a patch can be hidden in a middle of many other "failed QA" that have no consequences in master.
the more I think of it the more I suspect it would be a good idea to have a new status. -- Paul POULAIN http://www.biblibre.com Expert en Logiciels Libres pour l'info-doc Tel : (33) 4 91 81 35 08