Comment 15 for bug 1013786

Revision history for this message
Kathy Lussier (klussier) wrote :

I'm just adding a note that we already have two signoffs on this code. Once the merge conflict is resolved, it probably should be tested on, but the code was waiting on a couple of things:

- Jason had said he was going to move the check to a self-contained routine as had previously been suggested by Galen. Given that the comment was made a year ago, I'm guessing Jason is no longer planning to do so.

- Galen had asked that somebody (Jeff? Previous tester?) write a test plan in either the bug report or the commit message. I would suggest that it be done by whomever resolves the merge conflicts.

Until that last issue is resolved, I'm going to remove the pullrequet tag. This is a bug/feature that gets a lot of attention on Bug Squashing Day because a lot of people want to see it make its way into the code, but we just need a few small things done to it before it makes its way in.