Code review comment for lp:~eeejay/mago/blacklist

Revision history for this message
Eitan Isaacson (eeejay) wrote :

Yeah, I think we agree. I will make those changes soon, and comment on this
request when done.

On Tue, Aug 18, 2009 at 9:32 AM, Javier Collado <
<email address hidden>> wrote:

> Uhm, now I see what you wanted to do. Maybe changing the name from
> blacklist to skip is a good idea
> because I saw the change as implementing the complementary functionality to
> the already implemented
> whitelist. I understand that it might be needed to add something in the xml
> to skip/ignore/... a
> test case.
> --
> https://code.launchpad.net/~eeejay/mago/blacklist/+merge/10271<https://code.launchpad.net/%7Eeeejay/mago/blacklist/+merge/10271>
> You are the owner of lp:~eeejay/mago/blacklist.
>

« Back to merge proposal