[ovirt-devel] [Vdsm] silence existing pylint errors

[Nir Soffer]

On Fri, Apr 21, 2017 at 11:19 AM Dan Kenigsberg <danken at redhat.com> wrote:

> As we all know, Python is dynamically typed, which requires good
> coverage by unit tests.
>
> As we all know, Vdsm unit tests' coverage is not good.
>
> We need all the help we can get in order to avoid typos leading to
> bugs. Nir's `make pylint-diff` would give some.


We can take this patch now, since it does not break the build on errors.

It is useful as is so people can run make pylint locally and fix their
errors.


> In order to have it
> merged, I have posted
>
> https://gerrit.ovirt.org/#/q/status:open+project:vdsm+branch:master+topic:pylint+owner:danken
> to silence existing pylint errors. I hope I did not silence any
> serious ones; please review this topic as soon as possible.
>

Thanks for this great work!

I think the last patch is too big, and there are some storage issues that
we can fix
now. Can you split by vertical? I would like to take over the the storage
part.

I sent this patch making the storage/monitor.py pylint clean:
https://gerrit.ovirt.org/75736

Nir
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170421/871b5808/attachment.html>