Thank you all for your responses. It seems from your answers that there is not a consensus on that, so my suggestion is to accept them both, since they are both pep8-legal.
What do you think? Leo On Thu, Dec 5, 2013 at 8:41 AM, Guewen Baconnier < [email protected]> wrote: > On 12/04/2013 06:42 PM, Sandy Carter wrote: > >> I personally like the 2nd, but the point I was raising was in the >> specific case of the __openerp__.py dictionary: >> >> {'name': 'Module Name', >> 'version: '1.0', >> ... >> } >> >> While it is perfectly pep8 legal, it looks odd. >> >> > __openerp__.py is a place where I definitely prefer the > > > {'name': 'Module Name', > 'version': '1.0', > } > > version than > > { > > 'name': 'Module Name', > 'version': '1.0', > } > > So, a matter of taste and color... > > We could also come up with: > > dict(name='Module Name', > version='1.0') > > ;-) > > -- > Guewen > > > _______________________________________________ > Mailing list: https://launchpad.net/~openerp-community > Post to : [email protected] > Unsubscribe : https://launchpad.net/~openerp-community > More help : https://help.launchpad.net/ListHelp >
_______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp
