I also prefer second one for dictionaries. Regards.
2013/12/4 Andreas Maertens <[email protected]> > I prefer second. > often it looks like: > > context = { > 'item1' : item1, > 'item2' : item2, > } > > this is much better to read in source. > > lists are different. they might be better like second when having long > written items. if having short items, one liners are okay. > but first from the dicts look weired. imho we should avoid thad. > > > Von Samsung Mobile gesendet > > > > -------- Ursprüngliche Nachricht -------- > Von: Nicola Riolini <[email protected]> > Datum: 04.12.2013 17:41 (GMT+01:00) > An: [email protected] > Betreff: Re: [Openerp-community] Style: dictionaries > > > both 4 me, but with the comma (,) after the second element, for don't > forget if I append other ones' > > On 04/12/2013 17:28, Francesco Apruzzese wrote: > > The second one is the best for me. > > Nella citazione in data mer 04 dic 2013 17:24:37 CET, Leonardo Pistone ha > scritto: > > Hi all, > I bring to your attention Sandy's review here: > > > https://code.launchpad.net/~camptocamp/stock-logistic-flows/6.1-add-mrp_open_lot_attributes-lep/+merge/196749 > > <https://code.launchpad.net/%7Ecamptocamp/stock-logistic-flows/6.1-add-mrp_open_lot_attributes-lep/+merge/196749><https://code.launchpad.net/%7Ecamptocamp/stock-logistic-flows/6.1-add-mrp_open_lot_attributes-lep/+merge/196749> > > according to pep8, two styles of indentation are possible. Take for > example a dictionary: > > {'name': 'spam', > 'color': 'brown' > } > > or > > { > 'name': 'eggs', > 'color': 'white' > } > > According to the pep8 document and to the pep8 checker, they both > work. I personally would accept both. > > What do you think? > > Best, Leo > > > _______________________________________________ > 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 > > > -- > <http://www.micronaet.it/> > > > nicola riolini > [area tecnica] > > [image: Concessionario Autorizzato Passepartout > Mexal]<http://www.micronaet.it/>micronaet s.r.l. > > sede operativa: > via iv novembre 264 > 25010 borgosatollo (bs) > > tel./fax +39 030 2501517 > > [image: Skype Status] call me skype (utente skype: rn.micronaet) > > ATTENZIONE: Il contenuto di questo messaggio e rivolto unicamente alle > persone cui è indirizzato e può contenere informazioni la cui riservatezza > è tutelata legalmente. Ne sono vietati la riproduzione, la diffusione e > l'uso in mancanza di autorizzazione del destinatario. Se l'avete ricevuto > per errore vogliate eliminare il messaggio in modo permanente e darcene > cortesemente notizia. Analizzate sempre gli allegati prima di aprirli. > > > > ____________ > Virus checked by G Data AntiVirus > Version: AVA 22.14249 dated 04.12.2013 > Virus news: www.antiviruslab.com > > _______________________________________________ > 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
