[openstack-dev] [Ceilometer][Metering] Should Ceilometer adopt the HACKING guidelines?

Doug Hellmann doug.hellmann at dreamhost.com
Thu Jan 24 17:44:37 UTC 2013


On Thu, Jan 24, 2013 at 12:19 PM, Sean Dague <sdague at linux.vnet.ibm.com>wrote:

> On 01/24/2013 11:04 AM, Julien Danjou wrote:
>
>> On Thu, Jan 24 2013, Sandy Walsh wrote:
>>
>>  https://github.com/openstack/**nova/blob/master/HACKING.rst<https://github.com/openstack/nova/blob/master/HACKING.rst>
>>>
>>> It makes code reviews easier when coming in from other projects.
>>>
>>
>> Yes -- though I'll be very reticent to lose my time ordering import
>> lines alphabetically manually just because pep8 doesn't catch that
>> itself.
>>
>
> Nova has a hacking.py that does that check (and most of what's in
> HACKING.rst).
>
> Once we get a few more of the issues shaken out with it, it's going into
> oslo, so will be easy for projects to just sync from there.


Cool. Maybe we could even release it as a stand-alone tool so we can just
put it in our test-requires list. ;-)

Doug


>
>
>         -Sean
>
> --
> Sean Dague
> IBM Linux Technology Center
> email: sdague at linux.vnet.ibm.com
> alt-email: sldague at us.ibm.com
>
>
>
> ______________________________**_________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.**org <OpenStack-dev at lists.openstack.org>
> http://lists.openstack.org/**cgi-bin/mailman/listinfo/**openstack-dev<http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130124/e4a78110/attachment.html>


More information about the OpenStack-dev mailing list