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.