openerp-community team mailing list archive

[Leonardo Pistone <leonardo.pistone@xxxxxxxxxxxxxx>]

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 <
guewen.baconnier@xxxxxxxxxxxxxx> 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     : openerp-community@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openerp-community
> More help   : https://help.launchpad.net/ListHelp
>