Code review comment for lp:~gmb/launchpad/bug-999554

Revision history for this message
Graham Binns (gmb) wrote :

On 16 May 2012 16:17, Raphaël Badin <email address hidden> 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.

« Back to merge proposal