Code review comment for lp:~milo/lava-tool/lava-167

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

All the stuff around testing. You'd better just depend on py.test or something like that, it has that exact stuff integrated and ready. You may also find that just blindly enabling pyflakes and pep8 will throw lots of errors at you (unless lava is perfect, I doubt that). The better approach would be to use a diff mode and only run that at merge request time. Then you at least don't introduce _new_ issues.

« Back to merge proposal