On 16 May 2012 16:17, Raphaël Badin <[email protected]> wrote:
> Looks good to me.  I would have preferred a proper unit test instead of a doc 
> test but I guess it would be too much of a hassle.

Right. Given the context and the timing I stuck with appending
something rather than re-writing what was there (which probably would
have been pretty cheap, but I was being lazy). As it happens, this
View should go away at some point anyway, with the help of external
contributors.

-- 
https://code.launchpad.net/~gmb/launchpad/bug-999554/+merge/105944
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to