openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #01169
Re: [Merge] lp:~lin-yu/purchase-wkfl/add_purchase_price_list into lp:purchase-wkfl
Review: Needs Fixing code review
Thanks for your contribution Yu.
l163: please use orm.Model instead of osv.osv. Update the import accordingly.
l208, 463: no need to instantiate
l417: 2 lines are enough to comply with PEP8. Run flake8 to check for PEP8 issues
--
https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_price_list/+merge/180796
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.