openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #00841
Re: [Merge] lp:~the-clone-master/oemedical/oemedical into lp:oemedical
> Hi, Mario, thank you very much for your contribution. New import format is
> welcome, but why are you putting "import openerp" statement? This is not
> needed.
>
I will remove them right away...
> You can also improve another coding conventions, like replace osv.Model (or
> osv.osv if any) by orm.Model, put in __init__.py "from . import xxx" for local
> imports, and also PEP8 is desirable.
>
Will get them on next round... ;-)
--
https://code.launchpad.net/~the-clone-master/oemedical/oemedical/+merge/193037
Your team OEmedical Commiter is subscribed to branch lp:oemedical.
References