Code review comment for lp:~termie/nova/move_tests

Revision history for this message
Soren Hansen (soren) wrote :

On Mon, Jul 26, 2010 at 10:18:59PM -0000, Monty Taylor wrote:
> I can make good arguments for either... my personal preference is out
> though. My arguments for out:
>
> When I run pep8/pylint, I run it on nova/ ... if you have tests out,
> then you don't have to keep them clean. :)
>
> They don't get installed.

To me, both of those are good reasons why we should keep them in. :)

« Back to merge proposal