Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: OpenStack: Netstack

HACKING.rst

 

 

OpenStack netstack RSS feed   Index | Next | Previous | View Threaded


mnewby at internap

Apr 9, 2012, 8:39 PM

Post #1 of 1 (215 views)
Permalink
HACKING.rst

I have submitted a change that adds HACKING.rst, ala glance and nova, in the interest of promoting a coding style that is consistent with the other core projects.

https://review.openstack.org/#change,6395

Please participate in the review, if only to +1 it! Suggestions are, of course, welcome. The guidelines will ideally be used to evaluate code submissions under review, and therefore all quantum developers will be subject to them.

As you may have seen on the main OS mailing list, nova and glance do have some differences of opinion:

- nova, unlike glance, requires that only modules be imported, and that only one module be imported per line
- glance specifies slightly different docstring contentions

https://github.com/openstack/nova/blob/master/HACKING.rst
https://github.com/openstack/glance/blob/master/HACKING.rst

The version I've submitted is from nova with the module import rules stripped out (since the codebase doesn't currently conform to them). I personally like nova's 'only import modules' rule, since it ensures that circular imports don't creep in unintentionally, but I would like to see consensus that quantum developers are willing to pay the price of increased code verbosity. Given the difference of opinion that exists between glance and nova, there is clearly room for debate on the issue.

Cheers,


Maru
--
Mailing list: https://launchpad.net/~netstack
Post to : netstack [at] lists
Unsubscribe : https://launchpad.net/~netstack
More help : https://help.launchpad.net/ListHelp

OpenStack netstack RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.