On 04/04/2014 11:05, Sebastien Beau wrote: > Hi All, > I always hesitate to move or not on CamelCase for every class in OCA > module. > I prefer CamelCase but a lot of code do not use it. > And sometime not using CamelCase can introduce bug and you have to > rename the class in something different like > here > https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-change-class-name-to-avoid-error_rde/+merge/213881 > <https://code.launchpad.net/%7Ecamptocamp/carriers-deliveries/7.0-change-class-name-to-avoid-error_rde/+merge/213881> > > Maybe it's time to move on CamelCase and update the great Nicolas > Bessi tools to check the class name > (https://github.com/nbessi/openerp-conventions)? > > What do you think? > > > > _______________________________________________ > Mailing list: https://launchpad.net/~openerp-community > Post to : [email protected] > Unsubscribe : https://launchpad.net/~openerp-community > More help : https://help.launchpad.net/ListHelp
People will have to change their grepping habits, and it means a significant change in coding style between OCA addons and the official addons. -0 for me -- Alexandre Fayolle Chef de Projet Tel : + 33 (0)4 79 26 57 94 Camptocamp France SAS Savoie Technolac, BP 352 73377 Le Bourget du Lac Cedex http://www.camptocamp.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
