Igor, Personally this idea looks really nice to me, as this will help to avoid strange code being merged and not found via reviewing process. Cheers, Dina On Fri, Oct 3, 2014 at 12:40 PM, Igor Degtiarov <idegtiarov at mirantis.com> wrote: > Hi folks! > > I try too guess do we need in ceilometer checking new patches for > critical errors with pylint? > > As far as I know Nova and Sahara and others have such check. Actually > it is not checking of all project but comparing of the number of > errors without new patch and with it, and if diff is more then 0 then > patch are not taken. > > I have taken as pattern Sahara's solution and proposed a patch for > ceilometer: > https://review.openstack.org/#/c/125906/ > > Cheers, > Igor Degtiarov > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev at lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -- Best regards, Dina Belova Software Engineer Mirantis Inc. -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20141003/ea2a4965/attachment.html>