<div dir="ltr"><div>Igor,</div><div><br></div>Personally this idea looks really nice to me, as this will help to avoid strange code being merged and not found via reviewing process.<div><br></div><div>Cheers,</div><div>Dina</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 12:40 PM, Igor Degtiarov <span dir="ltr"><<a href="mailto:idegtiarov@mirantis.com" target="_blank">idegtiarov@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks!<br>
<br>
I try too guess do we need in ceilometer checking new patches for<br>
critical errors with pylint?<br>
<br>
As far as I know Nova and Sahara and others have such check. Actually<br>
it is not checking of all project but comparing of the number of<br>
errors without new patch and with it, and if diff is more then 0 then<br>
patch are not taken.<br>
<br>
I have taken as pattern Sahara's solution and proposed a patch for ceilometer:<br>
<a href="https://review.openstack.org/#/c/125906/" target="_blank">https://review.openstack.org/#/c/125906/</a><br>
<br>
Cheers,<br>
Igor Degtiarov<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"><p style="font-size:small;margin:0px;font-family:Helvetica">Best regards,</p><p style="font-size:small;margin:0px;font-family:Helvetica">Dina Belova</p><p style="font-size:small;margin:0px;font-family:Helvetica">Software Engineer</p><p style="font-size:small;margin:0px;font-family:Helvetica">Mirantis Inc.</p></div></div>
</div>