Server Environment And Tools

Code review comment for lp:~sylvain-legal/server-env-tools/7.0-auth_admin_passkey

  1. 7.0-auth_admin_passkey
  2. Merge into 7.0
Revision history for this message
Sylvain LE GAL (GRAP) (sylvain-legal) wrote :

Hi Pedro,

Thanks for your review.
I did modifications about import & pep8.

About description in __openerp__.py file : In other modules, there is a big header with that kind of informations (company name, date, link to AGPL licences, people who wrote the module).
In my modules, I just put a light header in each file as you seen and put all the information in the description field.
So the number of lines is smaller and information are more visible. for exemple if someone has a question about a module he can contact the authors without downloading the module.

Regards.

« Back to merge proposal