<div dir="ltr">Hi all,<div><br></div><div>Dan merge today a patch adding pylint-diff make target. With this </div><div>target, you can check new code quickly:</div><div><br></div><div><div>        $ time make pylint</div><div>        ...</div><div>        real 1m31.133s</div><div>        user 1m29.934s</div><div>        sys 0m0.849s</div><div>    </div><div>        $ time make pylint-diff</div><div>        ...</div><div>        real 0m12.364s</div><div>        user 0m11.879s</div><div>        sys 0m0.262s</div></div><div><br></div><div>Here is example patch adding stupid code to vdsm:</div><div><br></div><div><div><div><font face="monospace">diff --git a/lib/vdsm/common/osutils.py b/lib/vdsm/common/osutils.py</font></div><div><font face="monospace">index 0b3576a..6d37b7a 100644</font></div><div><font face="monospace">--- a/lib/vdsm/common/osutils.py</font></div><div><font face="monospace">+++ b/lib/vdsm/common/osutils.py</font></div><div><font face="monospace">@@ -55,5 +55,5 @@ def uninterruptible(func, *args, **kwargs):</font></div><div><font face="monospace">         try:</font></div><div><font face="monospace">             return func(*args, **kwargs)</font></div><div><font face="monospace">         except EnvironmentError as e:</font></div><div><font face="monospace">-            if e.errno != errno.EINTR:</font></div><div><font face="monospace">+            if e.errrno != errno.EINTR:</font></div></div></div><div><br></div><div>We had real errors like this in vdsm, some of them were made by the</div><div>author of this message :-)</div><div><br></div><div>Once committed, running pylint-diff will show:</div><div><br></div><div><div><font face="monospace">$ make pylint-diff</font></div><div><font face="monospace">[stripped uninteresting output]</font></div><div><span style="font-family:monospace">************* Module vdsm.common.osutils</span><br></div><div><font face="monospace">E: 58,15: Instance of &#39;EnvironmentError&#39; has no &#39;errrno&#39; member (no-member)</font></div><div><font face="monospace">ERROR: InvocationError: &#39;/home/nsoffer/.local/bin/pylint --errors-only lib/vdsm/common/osutils.py&#39;</font></div><div><font face="monospace">__________________________________________________________________________________________ summary ___________________________________________________________________________________________</font></div><div><font face="monospace">ERROR:   pylint: commands failed</font></div><div><font face="monospace">Makefile:978: recipe for target &#39;pylint-diff&#39; failed</font></div><div><font face="monospace">make: *** [pylint-diff] Error 1</font></div></div><div><br></div><div>This is not part of the build yet, since we still have lot of pylint errors</div><div>in vdsm. Hopefully we will be pylint clean this week, and we can enable</div><div>pylint during the build next week:</div><div><a href="https://gerrit.ovirt.org/75895">https://gerrit.ovirt.org/75895</a></div><div><br></div><div>Please run pylint-diff before you submit patches.</div><div><br></div><div>Nir </div></div>