Code review comment for lp:~bjornt/landscape-client/apt-facade-changer-fixes

Revision history for this message
Björn Tillenius (bjornt) wrote :

On Fri, Nov 11, 2011 at 03:28:08PM -0000, Alberto Donato wrote:
> Review: Approve
>
> Looks good! +1
>
> #1:
> + def test_mark_upgrade_no_upgrade(self):
> + """
> + If the candidate version of a package already is installed,
>
> typo, should be "is already installed".

Fixed.

--
Björn Tillenius | https://launchpad.net/~bjornt

« Back to merge proposal