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 <idegtia...@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@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



-- 

Best regards,

Dina Belova

Software Engineer

Mirantis Inc.
_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to