On 02/28/2014 08:56 PM, Jonas Sicking wrote:
On Feb 26, 2014 4:01 PM, "Jim Porter" <[email protected]> wrote:
If you don't have tests, but the patch should have tests, set
in-testsuite to "?" and file a followup to track landing the tests. When
they land, set in-testsuite to "+" for both bugs.
This should be very rare though. The only time when it is ok to land a
patch without tests is if it's for a critical blocker, we are close to
deadline, and if you've thoroughly tested the patch manually.
Yeah, let me be clear so that no one is confused: this "new" tracking
flag shouldn't affect whether you give a patch an r+ in any way
whatsoever. "in-testsuite?" on a FIXED bug isn't a way to allow more
patches to get r+'ed; it's just a way to keep track of the things we
were already doing anyway.
I hope that "in-testsuite?" on FIXED bugs will be exceedingly rare.
- Jim
_______________________________________________
dev-b2g mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-b2g
